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.

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

New spatially-variable horizontal upwind capability #122

Draft
wants to merge 49 commits into
base: master
Choose a base branch
from
Draft
Changes from 1 commit
Commits
Show all changes
49 commits
Select commit Hold shift + click to select a range
75f4052
Unify FM over all grids
tcclevenger Jan 8, 2025
bddf0cf
Fix rrtmgp standalone build
tcclevenger Feb 12, 2025
e0950ac
Create shortcut functions in FM with a single grid
tcclevenger Feb 12, 2025
3e77a07
No need to change name on grid clone
tcclevenger Feb 12, 2025
eb67ba6
Remove redundant field_layout::get_dim_names
tcclevenger Feb 12, 2025
08db442
Remove irrelevent change
tcclevenger Feb 12, 2025
0ccfd19
Fix indexing for subfields
tcclevenger Feb 12, 2025
2bacb7d
remove unneeded "Preffered" bundling definition
tcclevenger Feb 12, 2025
7c8cbe7
Fix unit test creation of FM, add check in AtmosphereInput that FM GM…
tcclevenger Feb 13, 2025
bdd9e87
Allow to change grid name in field clone
tcclevenger Feb 13, 2025
e0f9284
Register trivial group for field requests which include a group
tcclevenger Feb 17, 2025
63a878b
Correct group expectation in unit tests
tcclevenger Feb 18, 2025
2a063ea
Match grid name to field names when cloning in MAM tracer data reader
tcclevenger Feb 18, 2025
dd9a2fd
Fix 2x defined var after rebase
tcclevenger Feb 19, 2025
e899d85
Another error fix after rebase
tcclevenger Feb 19, 2025
0469923
Fix the FieldGroupInfo for groups which span grids
tcclevenger Feb 20, 2025
1e06484
Fix variable name after rebase
tcclevenger Feb 20, 2025
47acef0
Only add field to FieldGroupInfo if it doesn't exist
tcclevenger Feb 20, 2025
6237c87
Convert bundling to bool
tcclevenger Feb 20, 2025
f678280
Remove option for FM registering incomplete FieldRequest
tcclevenger Feb 21, 2025
f5bfe4d
Remove grids_mgr_type typedef
tcclevenger Feb 21, 2025
808d40d
Use equivalent_layout in AbstractRemapper::create_layout to reduce code
tcclevenger Feb 21, 2025
7344384
Purge FieldManager::size()
tcclevenger Feb 21, 2025
e0b7581
FieldManager::init_fields_time_stamp all grids at once
tcclevenger Feb 21, 2025
ddf26b1
Simplify subview indices check
tcclevenger Mar 3, 2025
8f71fd8
Add break to avoid unecessary queries to bundled group
tcclevenger Mar 3, 2025
afd13b0
Add todo for investigating possibly unecessary code
tcclevenger Mar 3, 2025
f57a733
Remove incomplete FR from AD_UT
tcclevenger Mar 3, 2025
92102b0
Use correct name for input grid in mam's tracer reader
tcclevenger Mar 5, 2025
514759a
Change get_groups_info -> get_group_info
tcclevenger Mar 5, 2025
2c54e67
Remove FM::get_field_ptr and FM::get_field_id.
tcclevenger Mar 5, 2025
1f33c0f
Chane logic in OutputManager to differentiate between types of parame…
tcclevenger Mar 5, 2025
05c7265
Merge branch 'master' into tcclevenger/eamxx/single_fm_multiple_grids
tcclevenger Mar 6, 2025
ae98638
Change back to FieldTracking::get_groups_info
tcclevenger Mar 6, 2025
b946a9c
Fix equivalent_layout tensor call
tcclevenger Mar 6, 2025
74cc9ab
EAMxx: simplify yaml syntax to allow skipping restart for an output s…
bartgol Feb 26, 2025
49bb138
EAMxx: better handle RUN_TYPE=branch
bartgol Feb 26, 2025
b3ac5bc
EAMxx: fix compilation error in rrtmgp with certain compilers
bartgol Feb 26, 2025
7e0cb14
EAMxx: fix setup of output managers in case of no output
bartgol Feb 26, 2025
6a70f62
EAMxx: fix docs linting
bartgol Feb 26, 2025
8a8f9cb
Use small kernels as default for shoc/p3 for pm-gpu
ndkeen Mar 7, 2025
3431067
accidentally checked in with these options OFF and we want them ON
ndkeen Mar 7, 2025
7c43a73
Merge branch 'tcclevenger/tcclevenger/eamxx/single_fm_multiple_grids'…
tcclevenger Mar 10, 2025
c9cba71
Merge branch 'bartgol/eamxx/io-branch-run-fixes' (PR #7063)
bartgol Mar 10, 2025
c617ec5
Merge branch E3SM-Project/ndk/machinefiles/pm-gpu-make-SMALL_KERNELS_…
ndkeen Mar 10, 2025
b57fbcd
New spatially variable upwinding capability
cbegeman Jan 28, 2025
32b09e5
Set upwind factor in diagnostics every time step
cbegeman Jan 28, 2025
1e9b4d0
Change minimum column thickness for centered-weighting based on dryin…
cbegeman Jan 28, 2025
32afa1b
Update build files
cbegeman Mar 10, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Simplify subview indices check
  • Loading branch information
tcclevenger committed Mar 3, 2025
commit ddf26b193722577bf9bbb93a36b231bb01eba58c
7 changes: 3 additions & 4 deletions components/eamxx/src/share/field/field_manager.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -312,10 +312,9 @@ get_field_group (const std::string& group_name, const std::string& grid_name) co
std::sort(ordered_subview_indices.begin(),ordered_subview_indices.end());

// Check that all subview indices are contiguous
for (size_t i=0; i<ordered_subview_indices.size()-1; ++i) {
EKAT_REQUIRE_MSG(ordered_subview_indices[i]+1==ordered_subview_indices[i+1],
"Error! Non-contiguous subview indices found in group \""+group_name+"\"\n");
}
size_t span = ordered_subview_indices.back() - ordered_subview_indices.front() + 1;
EKAT_REQUIRE_MSG (ordered_subview_indices.size()==span,
"Error! Non-contiguous subview indices found in group \""+group_name+"\"\n");

group.m_bundle = std::make_shared<Field>(
parent_field->subfield(
Expand Down