Skip to content

Simplify M loading and support loading a smaller eph region than device#64

Open
jonaslb wants to merge 1 commit intotfrederiksen:masterfrom
jonaslb:small_eph
Open

Simplify M loading and support loading a smaller eph region than device#64
jonaslb wants to merge 1 commit intotfrederiksen:masterfrom
jonaslb:small_eph

Conversation

@jonaslb
Copy link
Collaborator

@jonaslb jonaslb commented May 26, 2021

This is a minor simplification - reading the coupling M was done in two places and I moved it to a single method instead. Simultaneously I had a Phonon.nc file where I wanted to save e-ph couplings for a region smaller than the entire device region. I added support for this through a new eph_orbs_in_dev variable in the nc-file (this is the only place that it shows - i didn't document it...).

@tfrederiksen
Copy link
Owner

Nice! Do you also have the modification (to Phonons I suppose) where eph_orbs_in_dev is written to the nc-file?

@jonaslb
Copy link
Collaborator Author

jonaslb commented May 26, 2021

This I did with an entirely custom script - ie not using the Phonons command.

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