-
Notifications
You must be signed in to change notification settings - Fork 5
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
doesn't automatically generate C_lut.npy upon install #17
Comments
Yeah I think that's how it has to be done. It's a pretty big file, right? It could probably be generated automatically on first import or something like that. |
ah ok - understood... yeah it generates an 8.1MB file so maybe it's best to leave it to the user to generate the colormap. Although, as you said, maybe it's a good idea to have this automatically generated the first time the module is imported. |
Yeah, or come up with a more space-efficient way to store it in the repo: #10 |
If you know how to do a post-install whatever for setup.py feel free to work on making it automatic. I don't have any experience with such things so I'll have to learn it. |
Not sure if this is the right place to mention this but I cloned this repo and installed it and it didn't automatically generate the C_lut.npy lookup table.
I had to go into complex_colormap/complex_colormap and run generate.py
I'm not sure if this is the intended behaviour or not.
The text was updated successfully, but these errors were encountered: