patch: proc_attr_grabber.R

[glitt13]

Additions

• Feat: Geospatial coordinate search for comid if the nwissite gage id returns NA but does return a coordinate.
• A quick-and-dirty example script attr_gen_camels.R to download usgs nhdplus attribute data for all CAMELS locations, using local paths.
• Integrate the check_attr_selection() function into standard processing by placing inside the newly created variable checker wrapper wrap_check_vars(). wrap_check_vars() also checks that the sublist inside Retr_Params$vars has appropriate names for attribute variable categories (e.g. usgs_vars and ha_vars).

Removals

Changes

• Fix: Assign gage_id to data.frame in long format before concatenating using rbindlist.

Testing

Screenshots

Notes

Todos

Checklist

• PR has an informative and human-readable title
• Changes are limited to a single goal (no scope creep)
• Code can be automatically merged (no conflicts)
• Code follows project standards (link if applicable)
• Passes all existing automated tests
• Any change in functionality is tested
• New functions are documented (with a description, list of inputs, and expected output)
• Placeholder code is flagged / future todos are captured in comments
• Visually tested in supported browsers and devices (see checklist below 👇)
• Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)
• Reviewers requested with the Reviewers tool ➡️

Testing checklist

Target Environment support

• Windows
• Linux
• Browser

Accessibility

• Keyboard friendly
• Screen reader friendly

Other

• Is useable without CSS
• Is useable without JS
• Flexible from small to large screens
• No linting errors or warnings
• JavaScript tests are passing

[bolotinl]

Approved, all processing steps worked fine. Suggestion is to add to the wiki a step for creating the file data/input/user_data_std/juliemai-xSSA/nldi_feat_juliemai-xSSA_prediction.parquet (see backlog on the RaFts project board)