Enrico/bug context

[em1208]

• Closes # (insert issue number)
• Executed pre-commit run --all-files with no errors
• The change is fully covered by automated unit tests
• Documented in docs/ as appropriate
• Added an entry to the CHANGES file

I fixed a bug related to unit conversion when using context. Without the single line change in pint/facets/context/registry.py calling compatible_units fails with KeyError: <UnitsContainer({'[mass]': 1, '[time]': -1})>.

I hope this change can be merged and released soon!

[codspeed-hq]

CodSpeed Performance Report

Merging #1944 will not alter performance

_{Comparing em1208:enrico/bug-context (73e208c) with master (3cc2d36)}

Summary

✅ 439 untouched benchmarks

[hgrecco]

If I understand correctly, the problem is that after adding definitions, the cache is not rebuilt. Right? I think I have a better solution. I will try to put it in another PR

[em1208]

@hgrecco Yes, the problem is that when the context is enabled the cache should be rebuilt otherwise currently an exception is raised.

[hgrecco]

@em1208 This is the proposal #1779