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

Au ml crosswalk #562

Open
wants to merge 25 commits into
base: develop
Choose a base branch
from
Open

Au ml crosswalk #562

wants to merge 25 commits into from

Conversation

hillarymarler
Copy link
Collaborator

Draft functions related to monitoring location/Assessment Unit crosswalks:

  1. TADA_GetAssessmentUnitCrosswalk - using ATTAINS webservices, creates a crosswalk between Assessment Units and all Monitoring Locations recorded for them in ATTAINS

  2. TADA_GetMonLocByOrgId - create a df of MonitoringLocationIdentifier and OrganizationIdentifier to help w/ finalizing user created crosswalk (not sure if this is necessary? I was using it to help create test crosswalks)

  3. TADA_GetProviderRef - using WQX web services to create a ref file for the ProviderName for each Organizations (used later on to build the URLs for each MonitoringLocation to link to the WQP site pages)

  4. TADA_UpdateProviderRef - update the provider ref csv file saved in the package

  5. TADA_UpdateMonitoringLocationsInATTAINS - use a user-supplied Monitoring Location crosswalk w/ the ATTAINS created crosswalk to create a df to update Monitoring Location info in ATTAINS. I have not yet set up this function to save as a csv. I am thinking that as we will likely be setting up a number of similar functions in the future a TADA_CreateCSV function might be useful to create for the simple batch import csvs (in constrast to the complex mod 3 Excel files, which will need the more complicated export and formatting code built into each of those functions)

fix typo causing errors
Bug fix, update message text
switched message to a stop rather than print if no correct org id is supplied,  draft function to create crosswalk between org id and ml id (maybe for dev use only?)
Updated data links section
reorganizing function steps to make more efficient (don't pull in ATTAINS data unless necessary)
use TADA_CheckColumns
Updates to function order for efficiency, styler updates
Styler updates and minor formatting revisions to correct check error messages
corrected return issue
changed activity relative depth ref table prefix name to it is internal to TADA package, not created from WQX web services, used csv instead of get functions in new functions, added new TADA_Get functions to maintenance
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.

1 participant