Skip to content
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

Merge latest mindlessgen features into gp3_ipea #84

Merged
merged 10 commits into from
Nov 18, 2024

Conversation

marcelmbn
Copy link
Member

See main for details.

marcelmbn and others added 10 commits November 8, 2024 10:11
* update docs, co-workers, pyproject.toml, remove typos

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

* update CHANGELOG, make case more consistent

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

* update mindlessgen.toml

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

* align defaults

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

* update reference default value

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

* update tests

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

* correct inconsistent boolean

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

---------

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>
…ion (grimme-lab#78)

* fix random number generation by incorporating default_rng

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

* update CHANGELOG.md

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

---------

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>
* remove deprecated tests

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

* remove random numpy seed as it's pointless now

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

---------

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>
* A new function to set a fixed charge value

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* latest changes to the fixed charge method

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* Fixed charge and element composition works test have to be added

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* More modular fixed charge routine

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* complete fixed charge routine

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* Removed an unnessesary import

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* more convenient function name

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* Worked in the suggested changes

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* Added a changelog entry

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* Changes a bug with an array type

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* removed left over print statements

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* Implemented requested changes

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* Added a new test to raise a ValueError

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

---------

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>
* implement atlist to ati conversion

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

* add conversion test

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>

---------

Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>
…b#80)

* bug fix were uhf was always 0 if a fixed charge were given

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* updated a test for the right uhf

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* get rid of redundant code within set_random_charge

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* using atlist as a variable in misscellaneous

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

---------

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>
Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>
grimme-lab#74)

* The .UHF file is now written if the molecule has a uhf larger than 0

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* Writes the .CHRG file if the charge is larger than 0

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* charge and uhf can be None

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

* Negative charge is now written aswell

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>

---------

Signed-off-by: Jonathan Schöps <s6jtscho@uni-bonn.de>
Signed-off-by: Marcel Müller <marcel.mueller@thch.uni-bonn.de>
@marcelmbn marcelmbn added the enhancement New feature or request label Nov 18, 2024
@marcelmbn marcelmbn self-assigned this Nov 18, 2024
@marcelmbn marcelmbn merged commit af67c88 into grimme-lab:dev/gp3_ipea Nov 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants