-
Notifications
You must be signed in to change notification settings - Fork 14
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
Make unit model hashable #186
Comments
Hello @tstorek , thanks for the detailed report, it makes sense to me. Can you describe a bit more what do you mean by "caching", and what kind of error you have received while trying to cach? It could be useful to other users, who later come into the same problem. But anyway, it is nice to hear from you! :) |
@djs0109 Python supports builtin caching of functions as long as all |
Is your feature request related to a problem? Please describe.
Currently, the units model is not hashable although the content should be immutable and, hence, frozen.
But due to the missing Config in the model the
__hash__
function is not generated by pydantic.Describe the solution you'd like
I would like to add the following content to the model:
Describe alternatives you've considered
Inherit, the Unit model and add a specific Config object. However, this may lead to inconssitency
Additional context
Use case:
I have a list ob sensors that also contains information about the unit.
For mapping measurements to the sepcific sensors I regularly must loop throuth this list. Therefore, caching such lookup can be wuite useful. Caching though requires hashing of the Objects in the list that I qant to operate on.
The text was updated successfully, but these errors were encountered: