-
Notifications
You must be signed in to change notification settings - Fork 20
FF16 Rainfall API #324
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
Merged
Merged
FF16 Rainfall API #324
Changes from 9 commits
Commits
Show all changes
23 commits
Select commit
Hold shift + click to select a range
d447215
Implemented functions to explose auxillary variables in R API
devmitch 157bb29
Resized auxillary vector to appropriate size
devmitch d64bc0d
Exposed auxillary variables in R API + basic test
devmitch 98433de
Fixed all typos of 'auxillary' to 'auxiliary'
devmitch 1443006
Spline added to FF16 environment with basic API exposed + tests
devmitch f63e296
fixed merge conflicts with upstream develop
devmitch 7011189
basic test for rainfall spline over run_scm
devmitch da576f1
Default rainfall spline of y = 0
devmitch 96c55e6
Removed depricated infiltration rate settings, updated documentation
devmitch 47bac04
some changes to testing with regards to tolerance
devmitch aec5d5b
fixed incorrect tolerance
devmitch ed4000a
fixed comp function for std::is_sorted + more specific testsasdasd
devmitch e9a4211
less lenient tolerance to pass windows tests
devmitch 23ff3de
Refactored FF16w rainfall splines in favour of more generic 'extrinsi…
devmitch 1604176
Ability to get extrinsic driver names in R + small refactoring
devmitch 518019e
fixed old typo in patch tests
devmitch 3335925
extrapolation setting in interpolator
devmitch b1e2a86
trying with .at() rather than operator[]
devmitch 0c3e266
Remove rainfall_spline; update Rcpp
aornugent 7facab1
removed stringstream in favour of classic std::string for error handling
dfalster ffda217
more useful error messages for extrapolation
dfalster e38a623
adding return statement in evaluate
dfalster 846c0d8
Removed comments + minor refactor
dfalster 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
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
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.
You're right that need x & y. I would rather pass in a single data frame than an array, so perhaps change this accordingly?
Uh oh!
There was an error while loading. Please reload this page.
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.
So single data frame for
FF15w_make_environment()argument which is split into two arrays to pass intorainfall_add_points()?