👋 Thank you for your draft pull request! Do you know that you can use [ci skip] or [skip ci] in your commit messages to skip running continuous integration tests until you are ready?

Not necessarily this particular example per se, but maybe include an example where a unit can have multiple physical types? I ran into this one in my downstream package:

>>> from astropy import units as u
>>> FLAM = u.def_unit(
...     'flam', u.erg / (u.cm**2 * u.s * u.AA),
...     format={'generic': 'FLAM', 'console': 'FLAM'})
>>> FLAM.physical_type
PhysicalType({'power density', 'spectral flux density wav'})

Not necessarily this particular example per se, but maybe include an example where a unit can have multiple physical types?

Agreed! I still have to write that part, as well as an example on dimensional analysis with PhysicalType instances.

As I did in perhaps a bad way, and @mhvk suggested an alternative in #11595, it would be great to also have a list of the valid physical types in the docs. I agree that @mhvk's proposal is more in line with current astropy conventions... No problems if you don't want to do it here, I'll tackle it at some point in #11595, but then it won't be in v4.3!

As I did in perhaps a bad way, and @mhvk suggested an alternative in #11595, it would be great to also have a list of the valid physical types in the docs. I agree that @mhvk's proposal is more in line with current astropy conventions... No problems if you don't want to do it here, I'll tackle it at some point in #11595, but then it won't be in v4.3!

never mind. @mhvk is tackling this already!