-
Notifications
You must be signed in to change notification settings - Fork 274
Update SDA Downscaling to allow sf #3431
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
Open
dlebauer
wants to merge
33
commits into
PecanProject:develop
Choose a base branch
from
dlebauer:ensemble_downscaling
base: develop
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.
Open
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
4ac829f
also accept sf pts object
dlebauer be310eb
update CHANGELOG.md
dlebauer c5ae7dc
changed SDA downscale argument names to match new functionality
dlebauer db36719
make document
dlebauer 6ec8c14
merge upstream develop; resolve changelog conflict
dlebauer fa087c2
Update modules/assim.sequential/R/downscale_function.R
dlebauer da96331
SDA_downscale_preprocess now expects Date objects,
dlebauer 7041d18
update SDA downscale docs
dlebauer 6af65d9
remove deprecated date handling
dlebauer 3411085
Update modules/assim.sequential/R/downscale_function.R
dlebauer e4ca8a6
Refactoring and debugging SDA_downscale_preprocess; update test cases
dlebauer 978254a
merge
dlebauer 4f3ff5b
Merge branch 'develop' into ensemble_downscaling
dlebauer e00da0a
add refactored ensemble_downscale function derived from SDA_downscale…
dlebauer 44bcddc
reverted original downscale_funciton.R so it doesn't conflict with #3451
dlebauer f53eef0
Merge branch 'ensemble_downscaling' of github.com:dlebauer/pecan into…
dlebauer 0211780
Update CHANGELOG.md
dlebauer 6eee5ac
restore refactored SDA_downscale function
dlebauer 07daf1c
Merge branch 'ensemble_downscaling' of github.com:dlebauer/pecan into…
dlebauer b33a161
fix borked find and replace
dlebauer 307228f
update documentation
dlebauer 019afe4
revised ensemble downscale
dlebauer bc202e0
export subset_ensemble and ensemble_downscale functions
dlebauer 0d6fa01
update metrics function
dlebauer e7655e2
Merge branch 'develop' of github.com:pecanproject/pecan into ensemble…
dlebauer 01def9a
explicitly export outlier.etector.boxplot so it isn't interpreted as …
dlebauer ecd4ff1
Merge branch 'main' of github.com:pecanproject/pecan into ensemble_do…
dlebauer 29ab4d2
increased logging and set seed in future_map for ensemble_downscale f…
dlebauer 0310d65
replace randomForest with ranger in ensemble_downscale for speed and …
dlebauer 0309308
update downscale_metrics function
dlebauer b268ece
merge
dlebauer bddbc99
Refactor ensemble_downscale function for improved clarity and error h…
dlebauer 8235d54
Merge branch 'develop' into ensemble_downscaling
infotroph File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,10 +6,10 @@ | |
| #' @description This function performs a simple outlier replacement on all the columns of dataframes inside a list | ||
| #' @return A list the same dimension as X, with each column of each dataframe | ||
| #' modified by replacing outlier points with the column median | ||
| #' @export | ||
| #' @export outlier.detector.boxplot | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I bet this is trying to prevent this name being treated as an S3 method, but I recommend renaming the function instead. Life's too short to fight with R about this. |
||
| #' @importFrom magrittr %>% | ||
| #' | ||
| outlier.detector.boxplot<-function(X) { | ||
| outlier.detector.boxplot <- function(X) { | ||
| X <- X %>% | ||
| purrr::map(function(X.tmp){ | ||
| #X.tmp is all the state variables for each element of the list (site) | ||
|
|
||
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nits:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also: Need to move up to "unlreased" (this section is now for already-tagged 1.9.0)