Skip to content

deps: update code for x_orientation for new pyuvdata 3.2#433

Open
steven-murray wants to merge 4 commits intomainfrom
correct-xorientation
Open

deps: update code for x_orientation for new pyuvdata 3.2#433
steven-murray wants to merge 4 commits intomainfrom
correct-xorientation

Conversation

@steven-murray
Copy link
Copy Markdown
Contributor

This updates access to the .x_orientation attribute in line with recent pyuvdata updates.

@steven-murray steven-murray added the dependencies Pull requests that update a dependency file label Apr 6, 2026
@steven-murray steven-murray self-assigned this Apr 6, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.11%. Comparing base (2af8420) to head (0628b43).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #433      +/-   ##
==========================================
- Coverage   96.14%   96.11%   -0.04%     
==========================================
  Files          19       19              
  Lines        6350     6351       +1     
==========================================
- Hits         6105     6104       -1     
- Misses        245      247       +2     
Flag Coverage Δ
unittests 96.11% <100.00%> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

@adeliegorce adeliegorce left a comment

Choose a reason for hiding this comment

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

Thank you for these changes that were long overdue @steven-murray. Will approve the PR once the tests pass (which is independent from the x_orientation changes I believe).

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Some version compatibility issues in the tests following the changes... :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants