geoschem / integrated_methane_inversion

Integrated Methane Inversion workflow repository.
https://imi.readthedocs.org
MIT License
26 stars 22 forks source link

Negative methane concentration in GEOS-Chem crashes IMI #118

Closed djvaron closed 5 months ago

djvaron commented 1 year ago

GEOS-Chem crashes when negative methane concentrations are encountered. This crashes the IMI.

It's a rare occurrence, but it can happen, for example, if a buffer element receives a strongly negative posterior scale factor leading to strong negative emissions and thus negative concentrations in the lowest model layer.

A fix could be to update GEOS-Chem to replace negative concentrations with a small value (e.g., 1e-20) and issue a warning.

github-actions[bot] commented 5 months ago

This issue has been automatically marked as stale because it has not had recent activity. If there are no updates within 7 days it will be closed. You can add the "never stale" tag to prevent the issue from closing this issue.

msulprizio commented 5 months ago

@djvaron Is this issue still occurring? If so, do you recall where in GEOS-Chem it's happening? I can add a fix in 14.3.1.

djvaron commented 5 months ago

@msulprizio I haven't encountered this problem since I first reported it. My solution was to reset the prior estimates for the buffer elements after each weekly inversion in the Kalman filter framework (as we had done previously).

Have there been any updates to handle negative methane concentrations in GC or do they still crash the model? Crashing seems reasonable to me since encountering negative concentrations due to a very strong buffer element scale factor would suggest the boundary conditions are deficient.

msulprizio commented 5 months ago

We did not end up making a change to allow for negative methane concentrations in GEOS-Chem, so the model should still crash if negative values are encountered. I will close out this issue for now since we want to keep this behavior.