Skip to content
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

Fix timeseries1_weekly.h5 to work with pandas v1.1.0 #917

Merged
merged 1 commit into from
Aug 10, 2020
Merged

Conversation

jetuk
Copy link
Member

@jetuk jetuk commented Aug 7, 2020

Recent versions of pandas were unable to load this file. The
file has been recreated from the equivalent CSV with a 7D
frequency.

Also tidied up a few unneeded attributes in the associated test
models.

Fixes #916.

Recent versions of pandas were unable to load this file. The
file has been recreated from the equivalent CSV with a 7D
frequency.

Also tidied up a few unneeded attributes in the associated test
models.

Fixes #916.
@jetuk jetuk requested a review from Batch21 August 7, 2020 07:58
@jetuk jetuk merged commit 6339a4b into master Aug 10, 2020
@jetuk jetuk deleted the fix916 branch August 10, 2020 10:13
@KevisPachos
Copy link
Contributor

Hi @jetuk,
How can I update my h5 file to be compatible with Pandas v.1.1.0?
For instance, I have a file with 180 flow scenarios for 5 catchments called catchmod_flows_by_catchment_sub200.h5 that you had shared with me in the past.
Can I make changes to the existing file? Or do I need to recreate it by changing something?
And if so, how do I do that?
Thanks!

@jetuk
Copy link
Member Author

jetuk commented Dec 11, 2020

@KevisPachos for this one I recreated it from the CSV data. The issue is here in pandas: pandas-dev/pandas#33186

The only work around is to open it in an old version of pandas, save to another format (e.g. CSV), read in new version of pandas and then save to HDF again.

@KevisPachos
Copy link
Contributor

@jetuk, oh I see. It is a matter of importing and exporting using different pandas version. This is where virtual environments will be handy :) Thanks James

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Test failures with pandas v1.1.0
3 participants