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.

Sign up for GitHub

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

Jump to bottom

Improve missing Column Error #52

Merged
merged 2 commits into from
Jan 21, 2025
Merged

Improve missing Column Error #52

merged 2 commits into from
Jan 21, 2025

Conversation

sjperkins
Copy link
Member

@sjperkins sjperkins commented Jan 21, 2025 •
edited by github-actions bot
Loading

Change the error to indicate that the column did not exist on the Measurement Set

Thanks for contributing to xarray-ms.

We would appreciate it if you could add:

  • Test Cases covering your PR.
  • Documentation.
  • A Changelog entry in doc/source/changelog.rst.

📚 Documentation preview 📚: https://xarray-ms--52.org.readthedocs.build/en/52/

@sjperkins sjperkins requested a review from JSKenyon January 21, 2025 08:28
@sjperkins sjperkins merged commit 849f6b5 into main Jan 21, 2025
1 check passed
@sjperkins sjperkins deleted the improve-missing-column-error branch January 21, 2025 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JSKenyon JSKenyon JSKenyon approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sjperkins @JSKenyon