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

[PULL REQUEST] Remove double counting of acetone from monoterpenes #93

Merged
merged 1 commit into from Sep 14, 2021

Conversation

dylanbm
Copy link
Contributor

@dylanbm dylanbm commented Jul 23, 2021

Signed-off-by: Dylan B. Millet dbm@umn.edu

Signed-off-by: Dylan B. Millet <dbm@umn.edu>
@yantosca yantosca changed the title Remove double counting of acetone from monoterpenes [PULL REQUEST] Remove double counting of acetone from monoterpenes Jul 23, 2021
@yantosca yantosca added category: Bug Something isn't working topic: HEMCO Extensions Pertaining to HEMCO extensions labels Jul 23, 2021
@yantosca yantosca added this to the 3.2.0 milestone Aug 4, 2021
@yantosca
Copy link
Contributor

yantosca commented Aug 4, 2021

This is an update that will affect science, so it will ship with HEMCO feature version 3.2.0.

Copy link
Contributor

@yantosca yantosca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a straightforward fix that removes the FLUXACETmo array, which was causing double-counting in the sum. OK for (eventual) merge.

@yantosca
Copy link
Contributor

yantosca commented Aug 4, 2021

This update has been merged into feature/MEGAN-bugfixes, which will be merged into main for HEMCO feature release 3.2.0.

@yantosca yantosca self-assigned this Aug 4, 2021
@stale
Copy link

stale bot commented Sep 3, 2021

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 Stale bot from closing this issue.

@stale stale bot added the stale No recent activity on this issue label Sep 3, 2021
@stale
Copy link

stale bot commented Sep 11, 2021

Closing due to inactivity

@stale stale bot closed this Sep 11, 2021
@msulprizio msulprizio reopened this Sep 12, 2021
@msulprizio
Copy link
Contributor

I am keeping this PR open. This is going into the next version of HEMCO (3.2.0), which will be included in GEOS-Chem 13.3.0.

@msulprizio msulprizio changed the base branch from main to dev September 14, 2021 16:47
@stale stale bot removed the stale No recent activity on this issue label Sep 14, 2021
@msulprizio msulprizio merged commit 53d142a into geoschem:dev Sep 14, 2021
yantosca added a commit to geoschem/GCClassic that referenced this pull request Jan 21, 2022
This commit informs the GCClassic superproject about the following
commits that were pushed to the Github geoschem/geos-chem repository:

f31ffd6 Final cleanup
570eb98 Use standard solution not GEOS solution for ensuring LevDct_Unit is defined
5a98989 Prevent undefined variable when calculating vertical scale factor
27093bd Merge branch 'dev' into feature/ewlundgr/hemco_3.2.2_in_geos
06dfdc8 Merge branch 'feature/no_more_empty_error_messages' into dev
d1fa54b Catch a few undefined uses of LOC
1377d0a Remove CMake command to enable OpenMP by default
cd41222 Merge remote-tracking branch 'upstream/main' into feature/ewlundgr/hemco_3.2.2
4ee0bd7 Few lines added to hco_types_mod.F90 and mom6_cmake/CMakeLists.txt for succesful build
141a414 remove VFEI code since not yet operational
3f76bd0 Add SO4 to VFEI
28e5393 Get soil NOx restart files on every run call rather than just at the beginning
53ab3f7 Declare HEMCO restart field as missing if all values are negative, rather than all zeros
6a58b8f Use internal day tracker instead of HEMCO clock for updating volcano emissions
e96c130 Add description and make PBL fraction and max. emission height a configurable parameter
0b211cf Add VFEI biomass burning extension
03c7b00 Merge branch 'geos/develop' of https://github.com/GEOS-ESM/HEMCO into geos/develop
a6a5e76 Updates in support of 3km GEOSchem run
fd0222b Make sure that HEMCO level unit is always defined
27ce8e8 Fast-track bug fix for biogenic acetone double-counting (geoschem/HEMCO#93)
7ede721 prevent undefined variable when calculating vertical scale factor
6bd123e Merge branch 'geoschem/3.0' into feature/elundgren/update_to_3.0
5365483 Merge pull request #6 from GEOS-ESM/bugfix/mathomp4/remove-redundant-files
8f2a4e8 HEMCO structural updates, including header name changes, build updates, and updates to precision settings.

This brings various updates that facilitate HEMCO running within
the NASA GEOS ESM.

Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
yantosca added a commit to geoschem/GCHP that referenced this pull request Jan 21, 2022
This commit informs the GCHP superproject about the following
commits that were pushed to the Github geoschem/geos-chem repository:

f31ffd6 Final cleanup
570eb98 Use standard solution not GEOS solution for ensuring LevDct_Unit is defined
5a98989 Prevent undefined variable when calculating vertical scale factor
27093bd Merge branch 'dev' into feature/ewlundgr/hemco_3.2.2_in_geos
06dfdc8 Merge branch 'feature/no_more_empty_error_messages' into dev
d1fa54b Catch a few undefined uses of LOC
1377d0a Remove CMake command to enable OpenMP by default
cd41222 Merge remote-tracking branch 'upstream/main' into feature/ewlundgr/hemco_3.2.2
4ee0bd7 Few lines added to hco_types_mod.F90 and mom6_cmake/CMakeLists.txt for succesful build
141a414 remove VFEI code since not yet operational
3f76bd0 Add SO4 to VFEI
28e5393 Get soil NOx restart files on every run call rather than just at the beginning
53ab3f7 Declare HEMCO restart field as missing if all values are negative, rather than all zeros    6a58b8f Use internal day tracker instead of HEMCO clock for updating volcano emissions
e96c130 Add description and make PBL fraction and max. emission height a configurable parameter
0b211cf Add VFEI biomass burning extension
03c7b00 Merge branch 'geos/develop' of https://github.com/GEOS-ESM/HEMCO into geos/develop
a6a5e76 Updates in support of 3km GEOSchem run
fd0222b Make sure that HEMCO level unit is always defined
27ce8e8 Fast-track bug fix for biogenic acetone double-counting (geoschem/HEMCO#93)
7ede721 prevent undefined variable when calculating vertical scale factor
6bd123e Merge branch 'geoschem/3.0' into feature/elundgren/update_to_3.0
5365483 Merge pull request #6 from GEOS-ESM/bugfix/mathomp4/remove-redundant-files
8f2a4e8 HEMCO structural updates, including header name changes, build updates, and updates to precision settings.

This brings various updates that facilitate HEMCO running within
the NASA GEOS ESM.

Signed-off-by: Bob Yantosca <yantosca@seas.harvard.edu>
@yantosca yantosca added category: Bug Fix Fixes a bug that was previously reported and removed category: Bug Something isn't working labels Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Bug Fix Fixes a bug that was previously reported topic: HEMCO Extensions Pertaining to HEMCO extensions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants