Skip to content

Conversation

@johannes-moegerle
Copy link
Contributor

@johannes-moegerle johannes-moegerle commented Nov 12, 2025

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds a new RydbergStateAlkalineJJ class to represent alkaline Rydberg states using JJ coupling, which is specifically intended for high-l (l ≥ 5) states where singlet and triplet quantum defects are nearly identical.

Key changes:

  • Introduced RydbergStateAlkalineJJ class for alkaline atoms with JJ coupling scheme
  • Added appropriate import for AngularKetJJ in the rydberg_state module
  • Exported the new class in the package's __init__.py

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
src/rydstate/rydberg_state.py Added new RydbergStateAlkalineJJ class (lines 307-380) with initialization, angular/radial properties, and quantum defect calculation. Also added AngularKetJJ import (line 11)
src/rydstate/__init__.py Added RydbergStateAlkalineJJ to imports and __all__ list to expose the new class in the public API

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@johannes-moegerle johannes-moegerle merged commit 8ded624 into main Dec 19, 2025
22 checks passed
@johannes-moegerle johannes-moegerle deleted the jj-coupling branch December 19, 2025 13:32
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