TRIQS / dft_tools

Interface to DFT codes
https://triqs.github.io/dft_tools
Other
40 stars 38 forks source link

advanced chemical potential finder & DC correction schemes #232

Closed alberto-carta closed 1 year ago

alberto-carta commented 1 year ago

Bunch of changes to improve compatibility with the Hartree Solver in solid_dmft

Changelog:

New methods to find chemical potential

New function to compute DC correction

Magnetic calculations Removed assertion checking for a non-magnetic calculation in quantum espresso when computing deltaN

alberto-carta commented 1 year ago

The tests seem to run on python 3.8, i used match case statements in my code which are available in the new python releases, would it be possible to upgrade to python 3.10?

alberto-carta commented 1 year ago

Removed case match statements in favor of an older python syntax

the-hampel commented 1 year ago

Thank you Alberto for these changes - I will rebase and merge this now.

opeil commented 1 year ago

I have updated my local repo with recent changes and my Python 3.7.7 choked on the self-documenting f-expression (e.g., line 1740) added in commit 0d25aefc7 of this PR (#232). Since this feature is introduced only in Python 3.8.x, I wonder what is the official requirement for the Python version? In the TRIQS docs in unstable the required version is stated as 3.6 (I am not even sure it supports f-strings at all). Should one update the documentation and set 3.8 as the minimum required version for the entire TRIQS?

On Tue, Mar 21, 2023 at 2:35 PM Alexander Hampel @.***> wrote:

Merged #232 https://github.com/TRIQS/dft_tools/pull/232 into unstable.

— Reply to this email directly, view it on GitHub https://github.com/TRIQS/dft_tools/pull/232#event-8805678904, or unsubscribe https://github.com/notifications/unsubscribe-auth/AA2PU7TNL6UDE5WWUCHP57TW5GVALANCNFSM6AAAAAAU6C4ABE . You are receiving this because you are subscribed to this thread.Message ID: @.***>

--

Dr. Oleg E. Peil Senior Scientist Materials Center Leoben Forschung GmbH Roseggerstrasse 12 A-8700 Leoben Austria Tel :+43 3842 45922 45

the-hampel commented 1 year ago

I just discussed with @Wentzell and we think it is reasonable to raise the Python version requirement for the next release to 3.8 (released 2019). Thank you for pointing this out. We will update the documentation and list for unstable the 3.8 version requirement. As far as I can see the f-strings are not in the 3.1.x release. We of course do not want to change the requirements of the current release. I hope you can find a workaround at this point.