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

Output Atm DAG file is incorrect #1869

Open
bartgol opened this issue Aug 17, 2022 · 2 comments · May be fixed by E3SM-Project/E3SM#6860
Open

Output Atm DAG file is incorrect #1869

bartgol opened this issue Aug 17, 2022 · 2 comments · May be fixed by E3SM-Project/E3SM#6860

Comments

@bartgol
Copy link
Contributor

bartgol commented Aug 17, 2022

This is not crucial, since it's only used for visual representation, but since we have it, we might fix it at some point.

Here's a DAG from an ne4np4 run of SCREAMv1 compset. There are self-dependencies (SC exporter), incorrect dependencies (cldfrac dep on SC exporter). Overall, the DAG also needs to be cleaned up. It was impl-ed in the early days of atm proc and field classes, so some assumption have changed, and the code can prob be made cleaner.

scream_atm_dag

@AaronDonahue
Copy link
Contributor

I'm guessing that @mjs271 's PR will fix this right? #3101

@bartgol
Copy link
Contributor Author

bartgol commented Dec 6, 2024

Yes.

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

Successfully merging a pull request may close this issue.

2 participants