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

Fix support for NDData objects with dask .data attributes #365

Merged
merged 1 commit into from
Jun 16, 2023

Conversation

astrofrog
Copy link
Member

Also remove some unused code

@astrofrog astrofrog added the bug label May 22, 2023
@codecov
Copy link

codecov bot commented May 22, 2023

Codecov Report

Merging #365 (d0e6926) into main (8ea36e7) will increase coverage by 0.02%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #365      +/-   ##
==========================================
+ Coverage   92.50%   92.52%   +0.02%     
==========================================
  Files          24       24              
  Lines         840      843       +3     
==========================================
+ Hits          777      780       +3     
  Misses         63       63              
Impacted Files Coverage Δ
reproject/utils.py 85.54% <100.00%> (+0.25%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@Cadair
Copy link
Member

Cadair commented May 22, 2023

Does this need a bug fix release for the DKIST data?

@astrofrog
Copy link
Member Author

Well it seemed to work without it but maybe worth checking if it improves performance

@astrofrog
Copy link
Member Author

(I don't think it will make much difference for your images)

@astrofrog astrofrog merged commit 9b9dde0 into astropy:main Jun 16, 2023
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants