Closed xiaoxiaozhu123 closed 3 months ago
Hi @xiaoxiaozhu123 , thank you for the detailed report and proposed solution! I will open an issue in monty
about this. (I am also a maggma
maintainer, so if it's appropriate to address there, I can make those changes, too).
Before I open the bug report, can you please share the exact versions of maggma
and monty
and python
that you have installed?
Hi @xiaoxiaozhu123 , thank you for the detailed report and proposed solution! I will open an issue in
monty
about this. (I am also amaggma
maintainer, so if it's appropriate to address there, I can make those changes, too).Before I open the bug report, can you please share the exact versions of
maggma
andmonty
andpython
that you have installed?
python 3.11.8
monty.__version__
>> '2024.2.26'
maggma.__version__
>> '0.64.0'
Hi @xiaoxiaozhu123 , thank you for the detailed report and proposed solution! I will open an issue in
monty
about this. (I am also amaggma
maintainer, so if it's appropriate to address there, I can make those changes, too).Before I open the bug report, can you please share the exact versions of
maggma
andmonty
andpython
that you have installed?
Hi. I wanted to follow up on an email I sent regarding a request for permission to utilize pyEQL's database within my project.
Would you be able to provide an update or any guidance on this matter?If you didn't receive the email, please let me know.
This is what I encountered recently when importing pyEQL on a Windows 10 platform with system language as Chinese:
This bug origins from function zopen (which traces back to monty.io.zopen), Line 715, mongolike.py, maggma libray. Maybe the developpers didn't test their packages on non-English platforms... I have "fixed" this bug by mannually modifying source code Line 715 to:
However it won't help other users who might face the same issue. I am neither a maggma nor a monty user so I decide to report this bug here. I suggest the developers rewrite moty.io.zopen to force accept kw-argument encoding='utf-8' before importing magma on pyEQL's initalization. Here is a demo code snippet (forgive me for showing the ugly lambda):