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

Removes few ELM fields that were added by default #6366

Merged
merged 1 commit into from
May 17, 2024

Conversation

bishtgautam
Copy link
Contributor

@bishtgautam bishtgautam commented Apr 24, 2024

The glacier-related fields should only be added when
create_glacier_mec_landunit = .true..

Fixes #6365

The glacier-related fields are only added when
`create_glacier_mec_landunit = .true.`.

Fixes #6365
@bishtgautam
Copy link
Contributor Author

@golaz Could you please try this branch to see if it fixes the error reported in #6365?

@bishtgautam bishtgautam requested a review from golaz April 24, 2024 17:16
@bishtgautam bishtgautam added Land bug fix PR non-BFB PR makes roundoff changes to answers. labels Apr 24, 2024
@bishtgautam
Copy link
Contributor Author

@rljacob While the answers will not change, the tests using this branch will report that the ELM restart file is missing a few fields. Should this PR be marked as BFB or non-BFB?

@bishtgautam bishtgautam changed the title Removes few ELM fields to be added by default Removes few ELM fields that were added by default Apr 24, 2024
@rljacob
Copy link
Member

rljacob commented Apr 24, 2024

restart files are not compared in testing. just history.

@bishtgautam bishtgautam removed the non-BFB PR makes roundoff changes to answers. label Apr 24, 2024
@bishtgautam
Copy link
Contributor Author

I have updated the label and comment.

@rljacob rljacob added this to the v3.0.1 milestone Apr 29, 2024
@rljacob
Copy link
Member

rljacob commented May 1, 2024

@golaz please try this out.

@peterdschwartz
Copy link
Contributor

peterdschwartz commented May 16, 2024

since @rljacob went ahead and approved, that I'll go ahead with the merge today unless there are any objections.

peterdschwartz added a commit that referenced this pull request May 16, 2024
The glacier-related fields should only be added when
create_glacier_mec_landunit = .true..

Fixes #6365
@peterdschwartz
Copy link
Contributor

Merged to next. Expect DIFFs due to FIELD LIST for any tests that compare elm.h0 files.

@peterdschwartz peterdschwartz merged commit d927e67 into master May 17, 2024
20 checks passed
@peterdschwartz peterdschwartz deleted the bishtgautam/lnd/fix-elm-restart branch May 17, 2024 18:11
@peterdschwartz
Copy link
Contributor

Merged to master. hopefully got all the DIFFs

@rljacob
Copy link
Member

rljacob commented Jun 13, 2024

@wlin7 will double-check that this fixes restart issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Latest master can't use v3.0.0 restarts
3 participants