generated from NOAA-OWP/owp-open-source-project-template
-
Notifications
You must be signed in to change notification settings - Fork 3
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
Attribute aggregation/transformation + plotting & evaluation analyses #34
Open
glitt13
wants to merge
90
commits into
main
Choose a base branch
from
dev
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ection() into standard processing
…ribute transforms
…rmation functions
…t transformation script's documentation
…at strings just-in-case user doesn't use f'{dir_base}'
…ssing comids or variables have been identified, else write message that there could be an issue in the logic
…: return a gdf of comids and coords rather than just comids when querying nhdplus; Co-authored-by: Lauren Bolotin <[email protected]> Co-authored-by: Guy Litt <[email protected]>
…dard train/test/evaluation processing
…adapt the comid retrieval to also return the geometry; feat: add train/test split alternative using specific comids for testing
…ig_grid, but rather algo_config
…turn format from test_fs_retr_nhdp_comids
…yle theme (#33) * Create custom matplotlib stylesheet for RaFTS plots * Flip axes on scatter; change perf to pred for clarity * Change perf to pred for clarity * Read in mplstyle file directly from fs_algo * incorporate plotting functions into fs_perf_viz.py * Use functions for creating file output paths * Change perf_map to pred_map --------- Co-authored-by: glitt13 <[email protected]>
21 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
An approach to aggregate and transform existing attribute data to create new attribute data
Additionally, create & save plots that visualize results and aid in algorithm performance
Additions
pca_stdscaled_tfrm
,plot_pca_stdscaled_tfrm
,plot_pca_stdscaled_cumulative_var
,std_pca_plot_path
, functions comprehensively summarized in theplot_pca_save_wrap
wrappersave_feat_imp_fig_wrap
wrapperAlgoEvalPlotLC
class, with functions comprehensively summarized with theplot_learning_curve_save_wrap
wrapperplot_pred_vs_obs_wrap
wrapperplot_map_pred_wrap
wrapperplot_best_algo_wrap
wrapper/scripts/analysis/fs_proc_viz_best_ealstm.py
and the more-formal/scripts/eval_ingest/ealstm/proc_ealstm_agu24.py
plus associated config files in theealstm/
directoryfs_proc_algo_viz.py
as an updated version offs_proc_algo.py
with new evaluation and plotting featuresRemovals
Changes
Testing
test_tfrm_attr.py
has been challenging to implement under a normal unittest package approach owing to a mysteriousdask.dataframe as dd
error. Implemented a work-around that partially tests this package by nixing most instances of using classes.Screenshots
Notes
Todos
Checklist
Testing checklist
Target Environment support
Accessibility
Other