-
Notifications
You must be signed in to change notification settings - Fork 477
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
use application reg in docs #1931
base: master
Are you sure you want to change the base?
Conversation
CodSpeed Performance ReportMerging #1931 will degrade performances by 20.71%Comparing Summary
Benchmarks breakdown
|
Some small changes to the text in docs/getting/tutorial.rst |
As I mentiones before, I like this change. I was discussing this with some friends which are not pint users. A lot of them ask "why get_application_registry?, Why so long?, Do you really need the application part of the name?" Shall we bikeshed this public API? (we can still leave |
pre-commit run --all-files
with no errorsreplaces
ureg = pint.UnitRegistry()
withureg = pint.get_application_registry()
in the docs