Skip to content

Added Natural Transition Orbital calculation and processing.#271

Closed
hughf2468 wants to merge 10 commits into
ispg-group:mainfrom
AaronH-sys:main
Closed

Added Natural Transition Orbital calculation and processing.#271
hughf2468 wants to merge 10 commits into
ispg-group:mainfrom
AaronH-sys:main

Conversation

@hughf2468
Copy link
Copy Markdown

Added steps to calculate NTOs when requested, convert to relevant ".cube" files, and compress these files.

Currently these files are stored in separate FolderData nodes (one for each conformer) and the PKs of these nodes are reported by each "NEA spectrum workflow". However, the code is compatible with a visualiser we have developed (https://github.com/AaronH-sys/atmospec-orbital-visualisation.git) with minor changes to enable it to accept multiple conformers if desired.

Requires "orca_plot" to be configured as an AiiDA code, as well as AiiDA shell and Cubehandler as additional dependencies. Cubehandler requires editing of its "pyproject.toml" to work under Python 3.9.

@danielhollas danielhollas self-requested a review April 27, 2026 09:53
Copy link
Copy Markdown
Member

@danielhollas danielhollas left a comment

Choose a reason for hiding this comment

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

Amazing work, congrats! 🎉

I just have a couple of quick comments. Obviously we don't have time to do full review but that's fine. Good luck with your submission!

Comment thread README.md
Copy-paste the following profile configuration (substitute path to ORCA and the Docker image name if needed)
```
port = 8888
default_apps = [ "aiidalab-ispg@git+https://github.com/ispg-group/aiidalab-ispg.git@main",]
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I know this has caused problems for you at the beginning but it works now so please revert this change.

Comment thread notebooks/atmospec.ipynb
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please run pre-commit to get rid of changes in this file:

The easiest way is perhaps to install uv package manager and then use

pip install uv
uvx pre-commit run --all-files

structures_to_trajectory,
)

from aiida_shell import launch_shell_job
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can you please add aiida-shell as a dependency in pyproject.toml?

(also need to add cubehandler later, but now we cannot due to the python version dependency). If you open a PR on cubehandler, you can add the dependency on your specific branch like this

cubehandler@git+https://github.com/<username>/cubhandler@<your_branch>

@danielhollas
Copy link
Copy Markdown
Member

Superseded by #272

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.

3 participants