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

Update solvers.py #206

Merged
merged 22 commits into from
Jul 25, 2024
Merged

Update solvers.py #206

merged 22 commits into from
Jul 25, 2024

Conversation

lajerm
Copy link
Collaborator

@lajerm lajerm commented Jun 5, 2024

rixs_1v1c_py:

  1. added new option for pol_type: 'isotropic'
  2. added optional argument skipGS to skip transitions to the ground state (direct peak)

lajerm added 7 commits June 5, 2024 10:15
rixs_1v1c_py:
1) added new option for pol_type: 'isotropic'
2) added optional argument skipGS to skip transitions to the ground state (direct peak)
Copy link
Contributor

@mpmdean mpmdean left a comment

Choose a reason for hiding this comment

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

Thanks @lajerm

here are some things to consider. All small stuff.

edrixs/solvers.py Outdated Show resolved Hide resolved
the angle (in radians) between the linear polarization vector and the scattering plane.

If str1 or str2 is 'isotropic' then the scattering is considered isotropic and the other
variables are ignored.
Copy link
Contributor

Choose a reason for hiding this comment

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

along each axis and the other variables are ignored.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Changed in the updated version.

edrixs/solvers.py Outdated Show resolved Hide resolved
edrixs/solvers.py Outdated Show resolved Hide resolved
edrixs/solvers.py Outdated Show resolved Hide resolved
edrixs/solvers.py Outdated Show resolved Hide resolved
edrixs/solvers.py Outdated Show resolved Hide resolved
edrixs/solvers.py Outdated Show resolved Hide resolved
edrixs/solvers.py Outdated Show resolved Hide resolved
lajerm and others added 15 commits July 23, 2024 16:25
if the elastic peak is ignored, the indices in gs_list are explicitly omitted from the set of final states, instead of the first len(gs_list) energy levels of eval_i.
@lajerm
Copy link
Collaborator Author

lajerm commented Jul 23, 2024

Suggestions implemented along with a small additional modification: In the case skip_gs=True, the indices of gs_list are removed from the list of final states instead of assuming that the ground states correspond to the first indices of eval_i.

@mpmdean
Copy link
Contributor

mpmdean commented Jul 24, 2024

Hi @lajerm @skarakuzu

I think we should go ahead and merge this

One note here, is that the wheels in checks > artifacts have file names like:

edrixs-0.post1+g1514542-cp312-cp312-linux_x86_64.whl

I thought that the part post1+g1514542 should be the version i.e. 1.1.0?

@mpmdean mpmdean merged commit 37d4870 into NSLS-II:master Jul 25, 2024
7 checks passed
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