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.

Sign up for GitHub

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

Jump to bottom

Diagnostic variable lumping #105

Closed
DrYool opened this issue Aug 9, 2023 · 1 comment · Fixed by #102
Closed

Diagnostic variable lumping #105

DrYool opened this issue Aug 9, 2023 · 1 comment · Fixed by #102
Assignees
@ledm

Comments

@DrYool
Copy link
Collaborator

DrYool commented Aug 9, 2023

Nitrate, Silicate, Temperature and Salinity are blended in the suite output page under the heading "N" near the bottom of the list of diagnostics shown. I don't know why that is, but I had the same issue last time I worked with BGCVal2. For instance:

https://gws-access.jasmin.ac.uk/public/esmeval/CompareReports/bgcval2/ayool/ukesm_oae_rate/
@ledm ledm self-assigned this Aug 9, 2023
ledm added a commit that referenced this issue Aug 9, 2023
@ledm
Solved #104 and #105
3feb196
@ledm ledm mentioned this issue Aug 9, 2023
Merged
@DrYool
Copy link
Collaborator Author

DrYool commented Aug 14, 2023

This looks fine now in the TerraFIRMA job. I'll need to update my installation to check that it's alright in my suites, but I'm pretty sure the fix will hold. Cheers.

@DrYool DrYool closed this as completed Aug 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ledm ledm

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

New boolean in analysis_compare to run report maker only valeriupredoi/bgcval2
2 participants
@ledm @DrYool