Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fetchNASIS("components"): use duplicates argument and custom profile ID for many:1 relationships of legend, mapunit to datamapunit #251

Merged
merged 3 commits into from
Jun 24, 2022

Conversation

brownag
Copy link
Member

@brownag brownag commented Jun 11, 2022

Many mapunits. Uses get_component_correlation_data_from_NASIS_db(), provides dropAdditional dropNonRepresentative and duplicates as optional arguments to fetchNASIS(). Creates a non-standard "coiidcmb" from the combination of lmapunitiid:muiid:dmuiid:coiid which is used as the result SPC ID column.

This seems to now be working as I intended after checking to make sure all joins have an appropriate ID on left and right hand side.

@dylanbeaudette
Copy link
Member

Good idea, I've always been annoyed with the previous approach. I'll do some testing when I get back.

 -  fix join of site data, diagnostics and restrictions
 - add mukey (alias of lmapunitiid) to `get_component_correlation_data_from_NASIS_db()`
@brownag brownag marked this pull request as ready for review June 22, 2022 22:12
@brownag brownag changed the title fetchNASIS("components"): ideas on many:1 relationships of legend, mapunit to datamapunit fetchNASIS("components"): use duplicates argument and custom profile ID for many:1 relationships of legend, mapunit to datamapunit Jun 22, 2022
@brownag
Copy link
Member Author

brownag commented Jun 24, 2022

This works as it should, and does not change historic default behavior.

Slight breaking change from 2.7.1 in that duplicates argument is now being required to load data from mapunit/legend via fetchNASIS("components') whereas recently a (possibly incomplete) mapunit and legend table would be joined to SPC result iif the tables were populated in selected set/local DB.

@brownag brownag merged commit 2af3ad6 into master Jun 24, 2022
@brownag brownag deleted the manyMapunits branch June 24, 2022 23:13
brownag added a commit that referenced this pull request Jun 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants