What does `registry=None` mean in custom formatter ? about pint HOT 5 CLOSED

First, sorry for breaking things. The move to the new Formatter delegate allows more flexibility so I think this annoyance will pay off. Let me answer each of your questions:

Wouldn't a Unit object always be related to a registry, even if it's the default one ?
The formatter also format unit like objects (i.e. dicts mapping names to exponents) which do not necesesary require a registry.

Also, the way register_unit_format is written, wouldn't registry always be None
Yes. I was planning to try a solution but there was no need until now.

The idea would be to make this function look and instantiate the registered formatters.

    def get_formatter(self, spec: str):
        if spec == "":
            return self._formatters["D"]
        for k, v in self._formatters.items():
            if k in spec:
                return v

        try:
            orphan_fmt = REGISTERED_FORMATTERS[spec]
        except KeyError:
            return self._formatters["D"]

        try:
            fmt = orphan_fmt.__class__(self._registry)
            spec = get_attr(fmt, "spec", spec)
            self._formatters[spec] = fmt
            return fmt
        except:
            return orphan_fmt

from pint.

and then also add the spec attribute to NewFormatter here

from pint.

On a second thought
spec = get_attr(fmt, "spec", spec) and adding the spec attribut might not be necessary.

from pint.

I think I understand, thanks! I think I can try to suggest these changes in a PR if that helps.

from pint.

That would be great. I have review the code, it is necessary to add spec = get_attr(fmt, "spec", spec) and adding the spec attribute to NewFormatter. Let me know if you need help.

from pint.