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

Branching ratios between hyperfine and fine basis #163

Merged
merged 3 commits into from
May 19, 2024

Conversation

dihm
Copy link
Contributor

@dihm dihm commented Jan 3, 2024

I had a problem recently that needed to solve a system with sublevels from a fine Rydberg state and a low-lying hyperfine state. ARC provides all the tools, but I found what I think are gaps in calculating transition rates.

This PR adds some companion functions to the existing getBranchingRatio (which only covers hyperfine->hyperfine transition rates). Ultimately these functions are simple enough that there is an argument that they aren't necessary. But I think it would be good to add them to ARC since working them out has some tricky aspects for the average ARC user:

  • the FS to FS one uses the currently undocumented getSphericalDipoleMatrixElement method, so replicating functionality requires reading source code
  • going between HFS and FS requires scaling factors that I would call non-obvious. For those who aren't experts in reducing matrix elements (like myself), it would be nice to have a vetted source of truth.

Speaking of vetted source of truth, it would be good for others to confirm this experimentalist has actually calculated things correctly. I also have no preference regarding syntax or coding style, I merely tried to follow conventions set by getBranchingRatio. Happy to make whatever changes deemed necessary.

@nikolasibalic nikolasibalic self-requested a review April 3, 2024 21:10
@nikolasibalic nikolasibalic added this to the v3.5.0 milestone Apr 3, 2024
@nikolasibalic nikolasibalic force-pushed the master branch 5 times, most recently from f1c3718 to 0fc6544 Compare May 15, 2024 22:09
Copy link
Owner

@nikolasibalic nikolasibalic left a comment

Choose a reason for hiding this comment

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

I would like to understand (2 * self.I + 1) factor in FS to HFS better than I do at the moment. As far as I can tell, comparing with different calculations, this should be ok. I will merge this, and if there are any remaining errors in this section, we will figure out with use.

@nikolasibalic nikolasibalic merged commit 1b93c94 into nikolasibalic:master May 19, 2024
2 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.

None yet

2 participants