-
-
Notifications
You must be signed in to change notification settings - Fork 263
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
[FEATURE]: Implement support for concurrent installation with vaab/colour
Pypi package.
#1221
Comments
colour
Pypi package.
I thought about this one and I think there is an easy way to solve it: So basically, our |
References vaab/colour#62 |
Assuming it works as intended, I'm fine with the suggested fix. Certainly not clean or pretty, but it would be an improvement over the current status quo. |
colour
Pypi package.vaab/colour
Pypi package.
Please let me know once you release a new version that contains the new feature. |
In December sometimes ideally, with Python 3.12 compat. |
Hi @Voultapher, We discussed about this with the other Colour developers and it was preferred to have an opt-in behaviour, what we do here is dodgy as hell. The warnings are admittedly annoying and I will add another environment variable to disable them in a future release.
Because it is opt-in, there won't be really other alternatives than either having the user to explicitly enable the injection or provide them wrappers that do so. Cheers, Thomas |
Description
Recently I ran into #958 while wanting to use manim and colour-science in the same Python project. The workaround is not great. I'd appreciate it if you could build an additional package and release that together with the coluor-science package. Suggested feature:
The text was updated successfully, but these errors were encountered: