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.

Sign up for GitHub

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

Jump to bottom

Add data utilities for usecase 1 #28

Merged
merged 21 commits into from
Jan 20, 2023
Merged

Add data utilities for usecase 1 #28

merged 21 commits into from
Jan 20, 2023

Conversation

SarahAlidoost
Copy link
Contributor

@SarahAlidoost SarahAlidoost commented Dec 21, 2022 •
edited
Loading

relates #16

  • see TODOs in changed files
  • data organizations need more explorations
  • ML model definition

@SarahAlidoost
add some functions to prepare data, add an example script
b516bdc
@SarahAlidoost
change usecase_id
802e6f8
@SarahAlidoost
add data_utils to init file
e7cb570
@SarahAlidoost
move pyPhenology data to the data_utils, organize ppo_daymet data, re…
e3d3503 
…factor and fix bugs
@SarahAlidoost
move data preparations to data_utils
bb5ef83
@SarahAlidoost
add adatset_id to options
13284be
@SarahAlidoost
run usecase1_example script for train and test
85c4255
@SarahAlidoost
add msg for raise err
5afdee1
@SarahAlidoost
clean example script, add some comments
50df1a5
@SarahAlidoost
fix data merging, refactor
a53dfff
@SarahAlidoost
return predictors as dataset
1583030
@SarahAlidoost
add raster prediction to the example script
91eeea1
@SarahAlidoost
fix the values of metric in example script
eae3fdb
@SarahAlidoost
fix the name of statistics
b4c2cb2
@SarahAlidoost
refactor organizing data, consider weather stat as features
1271dc0
@SarahAlidoost
rename location to samples
6bd2360
@SarahAlidoost
refactor mask_data
ee41df2
@SarahAlidoost
change data coverage, add some plots
c23dd16
@SarahAlidoost
fix a typo
af97ad1
@SarahAlidoost
change data, remove plots from example script
50430d4
@SarahAlidoost
add a nb for plots of usecase1
7624ce8
@SarahAlidoost SarahAlidoost marked this pull request as ready for review January 18, 2023 18:06
Peter9192
Peter9192 approved these changes Jan 20, 2023
View reviewed changes
Copy link
Collaborator

@Peter9192 Peter9192 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot Sarah, very useful! I'm not sure whether we'll actually further develop this into a library, so I'm not gonna weigh heavily on the quality. Merging it directly as a reference for future work.

@Peter9192 Peter9192 merged commit 71354bb into main Jan 20, 2023
@Peter9192 Peter9192 deleted the prepare_data branch January 20, 2023 10:17
@Peter9192 Peter9192 mentioned this pull request Feb 7, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Peter9192 Peter9192 Peter9192 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@SarahAlidoost @Peter9192