-
Notifications
You must be signed in to change notification settings - Fork 45
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
Compatibility with 'uncertainties'? #124
Comments
Yes, this would be nice. @hgrecco has been wanting to separate out Measurement from pint, which might mean you want a separate package for uncertainties with pandas. Probably worth waiting till that's done too - although that's been on the agenda for a while now. |
xref: #140 |
closing this as it's now possible with UncertaintyArray |
Since I still miss the possibility to write a |
some (maybe all?) of those accessors could be added to uncertainties-pandas / pint-pandas would it be worth doing something to return Measurement objects? |
I thought about this as well, but since it requires both, pint-pandas and uncertainty-pandas, it does not really fit into one of them. But I'd be happy to hear your thoughts, @andrewgsavage, in case you disagree. Cheers, |
There should be accessors for .n, .s for uncertainties-pandas that operate on floats, without units. I am less sure on whether the pint+uncertainties accessors should be in pint-pandas or elsewhere. I think hgrecco mentioned splitting measurements outside of pint but that was a while ago. What you have is fine for now, we can see how well this works for people - which will need adding links to docs so they can find it! |
Thanks, I will certainly add links to the corresponding documentations of pint-pandas and uncertainties-pandas and need to add some descriptions to clarify my wording and harmonise the latter. The focus of the last weekend was the creation of a working codebase and corresponding tests. I hope to find some time during the week to pick the mentioned items up. Moreover, I agree on your statement regarding the need for uncertainties-pandas accessors that allow to retrieve the |
Many thanks for your work on this package and especially for
pint
itself! It makes life much easier, and in order to make life more easy, I would like to request a feature.In my opinion the usage of the "pint" dtype for a column containing an
uarray
would be very handy, but there seems to be an issue with the__init__
method ofPintArray
. If I get it correctly, it creates anp.array
from the provided data and thus wants to convert theuarray
into floats, which causes the following error (I shortened the path and inserted [...]):Here some snippets to reproduce the case:
A compatibility between
PintArray
anduncertainties
, as fulfilled by theMeasurement
class inpint
would in my opinion be a huge improvement inpint-pandas
.The text was updated successfully, but these errors were encountered: