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

Deconflict and cleanup Harvest Carbon PR #4

Closed
2 of 3 tasks
glemieux opened this issue Jul 18, 2022 · 1 comment
Closed
2 of 3 tasks

Deconflict and cleanup Harvest Carbon PR #4

glemieux opened this issue Jul 18, 2022 · 1 comment

Comments

@glemieux
Copy link
Owner

glemieux commented Jul 18, 2022 •
edited
Loading

PR location: E3SM-Project#5043

The configuration files need to be changed from what Shijie originally specified for his test cases. This includes reverting the .gitignore change as well:

  • Revert configuration changes

Ryan noted that we should ask someone if its ok to add the HRV_DEADSTEMC_TO_PROD... that Shijie added to a set of testmods. It should be noted that these variable predated this pr, but were not in the history output generally. They have also been added to the fates only history output, so this may conflict with that test run. Perhaps we can add a fates-specific test including these variables?

  • Ask someone in e3sm about the v1bgc testmod changes and change as necessary.

The default harvest was set as the area fraction originally. Should this be reverted?

  • Ask someone about the default harvest option and change as necessary.
@glemieux
Copy link
Owner Author

Closed per E3SM-Project#5106

glemieux pushed a commit that referenced this issue Jan 31, 2023
@sshu88
Merge pull request #4 from rgknox/fatesluc_cbasedhrv_v2_allocationfixes
d215068 
fates-nutrient coupling api changes
glemieux pushed a commit that referenced this issue Jun 13, 2023
@ambrad
Crusher: Work around tcmalloc issue.
3791fab 
cee/15.0.0 with GPU MPI buffers can crash in a system lib like this:

#4  0x00007fffe159e35b in (anonymous namespace)::do_free_with_callback(void*, void (*)(void*)) [clone .constprop.0] () from /opt/cray/pe/cce/15.0.0/cce/x86_64/lib/libtcmalloc_minimal.so.1
#5  0x00007fffe15a8f16 in tc_free () from /opt/cray/pe/cce/15.0.0/cce/x86_64/lib/libtcmalloc_minimal.so.1
#6  0x00007fffe99c2bcd in _dlerror_run () from /lib64/libdl.so.2
#7  0x00007fffe99c2481 in dlopen@@GLIBC_2.2.5 () from /lib64/libdl.so.2
#8  0x00007fffea7bce42 in _ad_cray_lock_init () from /opt/cray/pe/lib64/libmpi_cray.so.12
#9  0x00007fffed7eb37a in call_init.part () from /lib64/ld-linux-x86-64.so.2
#10 0x00007fffed7eb496 in _dl_init () from /lib64/ld-linux-x86-64.so.2
#11 0x00007fffed7dc58a in _dl_start_user () from /lib64/ld-linux-x86-64.so.2
#12 0x0000000000000001 in ?? ()
#13 0x00007fffffff42e7 in ?? ()
#14 0x0000000000000000 in ?? ()

Work around this by using cee/14.0.3.
glemieux pushed a commit that referenced this issue Nov 6, 2023
@eclare108213
Merge pull request #4 from eclare108213/allocate_rsnow
dc3b176 
allocate rsnow for use in icepack
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@glemieux