Skip to content

Conversation

cphyc
Copy link
Member

@cphyc cphyc commented Jul 18, 2025

PR Summary

This changes the default latex representation of comoving axes from $\mathrm{Mpc}/(1+z)$ to being $\mathrm{cMpc}$, the latter being the convention, while the former - albeit correct - is quite confusing and cumbersome.

Happy to not merge this if this is deemed controversial.

@cphyc cphyc changed the title Represent comoving with a 'c' prefix Represent comoving with a c prefix Jul 18, 2025
@matthewturk
Copy link
Member

"DALL-E, make me an image of @cphyc walking away from something that is exploding, like a cool action hero."

@cphyc cphyc added refactor improve readability, maintainability, modularity enhancement Making something better workshop-2025 labels Jul 18, 2025
@neutrinoceros neutrinoceros added this to the 4.5.0 milestone Jul 23, 2025
@neutrinoceros
Copy link
Member

well, easy enough to revert of monkeypatch if anyone complains about it later, so I think this is safe to pull in now. Thanks !

@neutrinoceros neutrinoceros merged commit be6eda5 into yt-project:main Jul 23, 2025
12 of 13 checks passed
@neutrinoceros neutrinoceros removed the refactor improve readability, maintainability, modularity label Jul 23, 2025
@neutrinoceros
Copy link
Member

(for reference: I'm denying the "refactor" qualification as this has user-visible implications)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Making something better workshop-2025
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants