-
-
Notifications
You must be signed in to change notification settings - Fork 18.5k
HDFStore still corrupted reads with utf8 #7244
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
Comments
version info:
|
ok...feel free to have a look, AFAICT it correct decides it can't decode then uses the vectorized decode. Not sure if the error is that |
maybe make an example that has astype decodables in one section and then non-decodables. plus the example needs to always pass/fail (on these cases). of course that means have to figure WHY it is failing! |
This might be the issue here: numpy/numpy#3939 If this is the case, then may have to either detect when can use this or only do the vectorize (which is much slower, unfortunately); but better to be correct and slow then wrong and fast This might be related as well: PyTables 'fixes' this for a |
@wabu fix for this? |
still did not dig deep enough into this ... |
Is there a fix for this bug? |
Seems to still be open. Are you interested in working on it?
…On Mon, Jun 17, 2019 at 1:47 PM OmerJog ***@***.***> wrote:
Is there a fix for this bug?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#7244?email_source=notifications&email_token=AAKAOIW4LTLOCD6VEIS73DLP27L3LA5CNFSM4AP2FZW2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODX4DEZI#issuecomment-502805093>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAKAOIUH3K3D7YKC7C3TIITP27L3LANCNFSM4AP2FZWQ>
.
|
hdf5 read still fails to read back some utf8 encoded strings back correctly.
I written a test script that:
The hdfstore often fails, but the data is stored correctly inside the hdf5 file.
Here's an example:
see #6505 for a previous issue on the topic
The text was updated successfully, but these errors were encountered: