-
Notifications
You must be signed in to change notification settings - Fork 640
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
h5topng still gives invalid colormap error #1066
Comments
I can reproduce this. The |
Thanks for looking into this. Installing h5utils build number 1106 in another environment works for me too. |
I faced same issue. I created another environment for build number 1106 of |
@ChristopherHogan, is the problem with build 1106 and Conda fixed yet? |
Another workaround is to pass the absolute path of the colormap file to the |
@oskooi tried it yesterday and said it is not fixed yet. I'll look into it. |
Hi, this still seems to be an issue. Is there a workaround for using output_png in Conda/Notebook that actually works. Passing the absolute path of colormaps to output_png doesn't work. |
OK i managed to get it to run with an absolute path. For me, if I set sim.use_output_directory() with no arguments, it creates a folder "-out" which h5topng is interpreting as an input argument and throws it's help message instead of converting. The solution was to give use_output_directory an argument (folder name) and also give output_png the absolute path to colormaps. |
I have the same issue today. |
Update and request to reopen this issue, I created a new conda environment with only h5utils installed and I still wasn't able to run h5topng. IMO this is a major problem for trying to debug simulations in meep. |
The h5utils Conda package was updated two weeks ago to use an updated HDF5 package. Perhaps the reason you are not able to use the latest version of h5utils is because you have multiple HDF5 libraries installed in your system which is causing a conflict? |
Possible, this is on a supercomputer so I don't have root access. The only way I've installed hdf5 is as a requirement installation for the meep (mpich) installation using
There should be no other hdf5 installations on the instance. I know that there are hdf5 modules that can be loaded on the network but I don't have any modules loaded. |
On a fresh installation of meep (newest version, parallel meep) the hdf5 version that is installed is still 1.12.1. Installing pymeep-extras doesn't change the version of hdf5 installed either. @oskooi
|
Okay, what I found matches the original issue, so having a separate h5utils environment works as a temporary workaround. Installing pymeep after having installed h5utils downgrades both h5utils and hdf5 to the versions which I assume have the colormap issue still.
I would like to suggest not closing this issue until meep works with the updated hdf5/h5utils. |
Actually, the same issue happens when you use a separate conda environment with only the h5utils installed using conda. |
You need h5utils 1.13.2 to fix this. Probably the Conda packages have not been updated yet? See NanoComp/h5utils#16 for an explanation and a workaround |
I just updated the h5utils Conda package to 1.13.2 which should be available momentarily: https://anaconda.org/conda-forge/h5utils. I will also update the PyMeep Conda package with this updated dependency. |
Thanks Ardavan! Much appreciated :) |
Per #1015 and #739, I tried updating h5utils on both conda-forge and simpetus but still got the invalid colormap error (meep imports fine though):
$ h5topng -t 0:332 -R -Zc dkbluered -v -a yarg -A eps-000000.00.h5 ez.h5
Using colormap "dkbluered" in file "/home/xxx/miniconda3/envs/meep/share/h5utils/colormaps/".
h5topng error: invalid colormap file
However the files are in the directory:
~/miniconda3/envs/meep/share/h5utils/colormaps$ ls
BrBG autumn bone cool dkbluered gray hot inferno lines prism summer viridis yarg
RdBu bluered colorcube copper flag green hsv jet pink spring vga winter yellow
The conda list is as follows:
and which gives
$ which h5topng
/home/xxx/miniconda3/envs/meep/bin/h5topng
The text was updated successfully, but these errors were encountered: