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

Check if mass flux is top-down and flip accordingly to enable processed mass flux #462

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

yuanjianz
Copy link

@yuanjianz yuanjianz commented Nov 19, 2024

Name and Institution (Required)

Name: Yuanjian Zhang
Institution: WashU

Describe the update

Check if mass flux is top-down and flip accordingly to enable processed mass flux

This update will go into dev/14.6.0

@lizziel
Copy link
Contributor

lizziel commented Dec 5, 2024

@yuanjianz, will you also have a geos-chem PR to make run directory creation using processed GEOS-IT data an option? The current code lists it as not available.

printf "Pre-processed GEOS-IT data at C180 resolution is not yet available. Try again.\n"
https://github.com/geoschem/geos-chem/blob/6bbd81fae499fc2cb566722be1a38a1567bf6e4c/run/GCHP/createRunDir.sh#L406

@lizziel
Copy link
Contributor

lizziel commented Dec 5, 2024

Also pinging @yidant on if there are any updates for GCHP run directory creation for using processed GEOS-IT cubed sphere files.

@yuanjianz
Copy link
Author

Hi @lizziel, sure. I have a feature branch that enables processed cubed sphere GEOSIT in my forked repo and ready to do a PR. Do you want me to merge it into which branch? Also, noted that we have not finished our evaluation of processed cs GEOSIT due to some issues on Compute1. But I expect it to be soon.

@lizziel
Copy link
Contributor

lizziel commented Dec 6, 2024

Thanks @yuanjianz. You can target dev/14.6.0 for this. It is okay if we do not yet have all of the processed files ready.

@lizziel
Copy link
Contributor

lizziel commented Dec 6, 2024

Thanks for creating geoschem/geos-chem#2616. Is this PR that goes with it ready for review? It is still marked as draft.

@yuanjianz yuanjianz marked this pull request as ready for review December 7, 2024 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants