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

196 selection class #205

Merged
merged 35 commits into from
Mar 6, 2024
Merged

196 selection class #205

merged 35 commits into from
Mar 6, 2024

Conversation

mpound
Copy link
Collaborator

@mpound mpound commented Mar 4, 2024

This PR is for the functionality of the selection class, issue #196. It includes the Selection class, documentation of the code, test code in test_selection.py, and an example notebook developer/test_selection.ipynb. This code contains ONLY the implementation of the selection class, it does not contain how that class is used by GBTFITSLoad or ultimately flagging. Happy selecting!

mpound added 30 commits January 30, 2024 15:27
2. allow quantities in coordinates
3. generic check_type method (note: should be moved to core.py)
4. general debugging
Add recursion to check_type to handle when input is array-like.
Add select_channel.
Add placeholder for select_time.
@mpound mpound requested a review from astrofle March 4, 2024 20:13
@mpound mpound added this to the Q4 milestone Mar 4, 2024
Copy link
Collaborator

@astrofle astrofle left a comment

Choose a reason for hiding this comment

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

Looks great!

@mpound mpound merged commit b29861e into main Mar 6, 2024
14 checks passed
@mpound mpound mentioned this pull request Mar 6, 2024
@astrofle astrofle deleted the 196_selection_class branch September 3, 2024 19: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