Skip to content

Conversation

@NParsonsMO
Copy link
Contributor

@NParsonsMO NParsonsMO commented Dec 17, 2025

Changing the f-strings in the plot titles to use a dataset's alias instead of its name.


Before you get started

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

New or updated recipe/diagnostic

New or updated data reformatting script


To help with the number of pull requests:

@NParsonsMO NParsonsMO self-assigned this Dec 17, 2025
@NParsonsMO NParsonsMO linked an issue Dec 17, 2025 that may be closed by this pull request
@NParsonsMO NParsonsMO changed the title Amending plot f-strings Radiation budget: labelling plots with alias Dec 17, 2025
@NParsonsMO
Copy link
Contributor Author

NParsonsMO commented Dec 17, 2025

Output before change:
image

@NParsonsMO
Copy link
Contributor Author

Output after change:
image

@NParsonsMO NParsonsMO requested a review from ehogan December 17, 2025 15:34
@NParsonsMO
Copy link
Contributor Author

@ehogan I'm not sure if I should change the picture in the documentation? Or mention alias in there somewhere?

@valeriupredoi
Copy link
Contributor

@ehogan yet again GH notifies me of a PR that's being created by a MO bod, though I'm neither assigned nor requested as a reviewer - heck, am not even of an owner of the code that's being changed; I noticed this previously with your guys' work on RTW, do you folks at the MO have a config setting that says "ping V everytime we open a PR"? 🤣

@ehogan
Copy link
Contributor

ehogan commented Dec 17, 2025

@ehogan yet again GH notifies me of a PR that's being created by a MO bod, though I'm neither assigned nor requested as a reviewer - heck, am not even of an owner of the code that's being changed; I noticed this previously with your guys' work on RTW, do you folks at the MO have a config setting that says "ping V everytime we open a PR"? 🤣

🤣 Not to my knowledge! 🤪 I can't see why you would have been notified?! 🤯

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.

Amend radiation budget slightly to label plots with an alias

4 participants