Skip to content

Conversation

@VladimirMakhnev
Copy link

To calculate U-MRSF energy the new input word "umrsf=true" in tdhf was added (see input example).
Jacobi rotations are applied by default.

@karmachoi
Copy link
Contributor

Don't create a new directory of inputs. Also delete any input files of GAMESS. Also your OpenQP input files are for ROHF not UHF.

@Alireza-Lashkaripour Alireza-Lashkaripour added documentation Improvements or additions to documentation enhancement New feature or request feature Feature enhancements labels Nov 24, 2025
Copy link
Contributor

@Alireza-Lashkaripour Alireza-Lashkaripour left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We also need to add xyz file for ch4 or just add the geo to inputs.

@@ -0,0 +1,25 @@
# MRSF-TDDFT/BHHLYP/6-31G* Geom. Opt. of Thymine S1 State
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please correct the explanation line and match with the inputs setup.

@@ -0,0 +1,28 @@
# MRSF-TDDFT/BHHLYP/6-31G* Geom. Opt. of Thymine S1 State
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same for here!

@@ -0,0 +1,25 @@
# MRSF-TDDFT/BHHLYP/6-31G* Geom. Opt. of Thymine S1 State
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same for here.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove GAMESS inputs.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove this one too.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove unnecessary spaces between lines.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove this input folder and create one in example folders like others or just put your inputs on others folder. Also, please change the name of the inputs (mimicking other excising inputs) to make them more readable and create json file for each one so it can be checked with each run_test.

@karmachoi
Copy link
Contributor

The keyword should not be like umrsf=true.
It should be type=umrsf in tdhf group. This key should also check to see if type=uhf in scf group too.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request feature Feature enhancements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants