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

Pynta calculate IRC in different mode by user's choice #44

Closed
wants to merge 35 commits into from

Conversation

sakim8048
Copy link
Contributor

@sakim8048 sakim8048 commented Apr 11, 2024

Summary of modification

I have updated the way of running IRC by user's choice.
In pynta class from main.py, the keyword irc_mode is added. User can add this keyword in Pynta input script.
If irc_mode = "relaxed", irc is calculated with top half of the slab layers relaxed. irc_obj_dict updated accordingly.
If irc_mode = "fixed", irc is calculated with all slab layers frozen. irc_obj_dict updated accordingly.
If irc_mode, is not "relaxed" nor "fixed", IRC is not calculated. It will call MolecularTSEstimate_noIRC in tasks.py


Updated comments (04/15/2024)

  • I added logging.info() in main.py and setup_transition_states(). This will generate pynta.log file and dump info, which file and module via logging.info, firetasks are running. It is only implemented in setup_transition_states() at the moment as a test. But will be expanded throughout pynta. print() statement is still there to parse the output to job output file generated by slurm.
  • Made IRC_obj_dict as optional keyword for MolecularTSEstimate

@sakim8048 sakim8048 marked this pull request as draft April 15, 2024 02:49
@sakim8048 sakim8048 marked this pull request as ready for review April 15, 2024 15:27
Copy link
Contributor

@mjohnson541 mjohnson541 left a comment

Choose a reason for hiding this comment

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

Some comments.

pynta/main.py Outdated Show resolved Hide resolved
pynta/main.py Outdated Show resolved Hide resolved
pynta/tasks.py Outdated Show resolved Hide resolved
typo fixed (stupid uppercase)...

typo fix

delete empty IRC_obj_dict

collect_firework() added again
typo fixed (stupid uppercase)...

typo fix

delete empty IRC_obj_dict

collect_firework() added again

if irc_mode is not defined, define irc_mode = skip

define IRC_obj_dict key value in run_task

fix typo (case sensitive)

delete prints() and redundent info for logger

add IRC_opj_dict in optional_params

indentation fix
fix typo

add irc_mode to ts_task

typo fixed (stupid uppercase)...

typo fix

delete empty IRC_obj_dict

collect_firework() added again

define IRC_obj_dict key value in run_task

fix typo (case sensitive)

add IRC_opj_dict in optional_params

indentation fix

define IRC_obj_dict key value in run_task

fix typo (case sensitive)

add IRC_opj_dict in optional_params

indentation fix

make irc_obj_dict optional in MolecularTSEstimate

fix typo

add irc_mode to ts_task

typo fixed (stupid uppercase)...

typo fix

delete empty IRC_obj_dict

collect_firework() added again
@sakim8048 sakim8048 closed this Apr 20, 2024
@sakim8048 sakim8048 deleted the pynta-irc branch April 23, 2024 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants