-
Notifications
You must be signed in to change notification settings - Fork 1
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
WIP PR for getting CI working as desired #10
base: master
Are you sure you want to change the base?
Conversation
Looks like the env file changes need to be merged in order to be pulled into the build Note I skipped test_logical because it's failing. It looks like the cause is that logical.avro is not a valid avro file. |
@martindurant any idea where to look for resolving this error? It'll take me a good chunk of diving into this to understand what is the right way to fix this. |
The values seem of completely the wrong order, so it could be an int32->64 kind of thing. In pandas v2, more time types became allowed. |
sure, I'm fine with that. This looks like a bug in fastparquet. t should always be a dtype. I think it should just be |
Before going further, you might be interested by https://awkward-array.org/doc/main/reference/generated/ak.from_avro_file.html in awkward-array, which I was just made aware of. This respects the truly nested nature of avro and might be what you want, but you can also transform to arrow or pandas if the dataset is really tabular. |
Hmm, the reason I was updating uavro is to get intake-avro working. I wasn't familiar with awkward-array, but I bet I could drop it in to replace uavro in intake-avro. Thoughts? |
No description provided.