Summa initial NGen contribution #513
Open
+1,021
−187
Conversation
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
…ing submodule branch that works for all builds too. Eventually this will be the master branch
* commit '90f6fc9bfa742f63b3d9194843527169b6a0664c': (307 commits) Replace unused boost/bind.hpp that was generating warnings with std's <functional> NullForcingProvider: Drop now-unnecessary private empty vector Add missing 'override' annotation Refactor get_available_variable_names to return a span<string> rather than a concrete vector<string> Cleanup from review feedback Removed requirement for `"path"` in forcing config Adjusted example file to use no new models or capabilities Delete a redundant code line Revise Bmi_Multi_Formulation and remove some include headers from NullForcingProvider Remove unused include header names Remove presetting available forcings Remove is_param_sum_over_time_step function. Revised NullForcingProvider incorporating Matt's comments Revise config jason to be force-less. Null/Constant Forcing Provider cmake: default python support to ON Another couple typos Random typo fix Remove commented-out code Remove dead code ...
* commit '72f121a5d12762d59c3870efd6d0598f2286886c': Fix a CMakelist.txt error. Add unit test for logging and other revisions Implement a logging function cmake: set NetCDF support to default to OFF Update test routing config to alleviate pydantic errors (NOAA-OWP#654)
* 'master' of https://github.com/ashleymedin/ngen: routing should be on cpu_pool 36, accidental change for mac git ignore if run in main directory summa tag fixing test compiling lgar-c was not compiling correctly submod updating summa ignoring submodules making agree with ciroh making agree with ciroh branch submodule
* 'master' of https://github.com/ashleymedin/ngen: resetting master
This reverts commit ed88e46.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
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.
[Short description explaining the high-level reason for the pull request]
@jameshalgren
Additions
Removals
Changes
Testing
cmake_build/ngen data/catchment_data.geojson "celia" ./data/nexus_data.geojson "nex-26" ./extern/summa/summa/test_ngen/example_realization_config_w_summa_bmi.json
or
cmake_build/ngen data/catchment_data.geojson "cat-27" ./data/nexus_data.geojson "nex-26" ./extern/summa/summa/test_ngen/example_realization_config_w_summa_bmi__mac.json
This command can be run as ./extern/summa/summa/test_ngen/example_run_celia.[system_type].sh also.
Screenshots
Notes
Todos
Checklist
Testing checklist (automated report can be put here)
Target Environment support