-
Notifications
You must be signed in to change notification settings - Fork 15
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
Tests for Dataset.crop() #310
Merged
Merged
Changes from 2 commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
0bacfdf
Add fixture for new VISP asdf (remove this later too)
SolarDrew 07326bb
Add very basic Stokes crop test
SolarDrew 76248a7
Add another fixture for the new asdf
SolarDrew 10b8645
Add my own test env for tox because it was being a pain
SolarDrew a1a2301
Fix the new fixture
SolarDrew cb45d60
More cropping tests
SolarDrew 3467514
More variations on cropping tests
SolarDrew f8f1a68
Lots of changes to make the tests work
SolarDrew 85fe54b
Add an extra test
SolarDrew e6150fa
Trim out some extra testing that probably isn't useful any more
SolarDrew 6c41cfb
Correction and some more assertions
SolarDrew 8c5a7ed
Trim more stuff, add more assertions
SolarDrew 86448ad
Minor corrections
SolarDrew a7be3a5
Add test for cropping visp by time
SolarDrew 8c1a731
Clear out some unneeded comments
SolarDrew 1f8c859
Make latest test use user-defined coords and add assertions
SolarDrew c85c11f
Add test for cropping visp by lon/lat
SolarDrew 2393e9a
More testing. Always more testing
SolarDrew 03c88e6
Tweak a few numbers
SolarDrew fdf96be
Yet another assertion
SolarDrew fb7364c
Yes! It's more assertions!
SolarDrew 3001020
Add some plots for now, might remove later
SolarDrew 4cd2b09
Rebuild gwcs headers in files used for crop fixtures
SolarDrew 52fe302
Make the assertions pass
SolarDrew f8bc814
Don't need this
SolarDrew 27dccd6
Don't need plotting right now
SolarDrew c16f216
Bunch of coords need updating for fixed gwcs headers
SolarDrew 43194c8
Add dkist simulator and inventory to deps
SolarDrew af32415
Tidying and correct numbers
SolarDrew 7c755c0
Want this back after all
SolarDrew d983710
More assertions and tweak some numbers to make smaller arrays
SolarDrew 44eaa40
Add new (compressed) dataset asdf for testing crop
SolarDrew 6bf3f14
Update conftest for new compressed VISP test file
SolarDrew 9419de5
Add compressed cryo file for crop testing
SolarDrew 17b8da8
Update conftest for new cryo file
SolarDrew 27e6ec6
Tidy up tests a little
SolarDrew 9ed86f0
Don't need custom test env any more
SolarDrew ebfdbe5
Check Stokes headers of Stokes-cropped datasets
SolarDrew b211d59
Correct rebase fail
SolarDrew 50e4f9e
Merge branch 'main' of github.com:DKISTDC/dkist into bugfix-192
SolarDrew 5cb429b
Update tox.ini
SolarDrew 1b49a64
Merge branch 'main' of github.com:DKISTDC/dkist into bugfix-192
SolarDrew 9921256
Formatting
SolarDrew 6a4bd03
Brainfart - thank you @Cadair
SolarDrew 28fefa4
Skip dataset crop tests if gwcs isn't up to date
SolarDrew e94a5d0
Changelog
SolarDrew b052683
Update dkist/dataset/tests/test_crop_dataset.py
Cadair File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
Add tests for cropping datasets by world coords |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was thinking more in the test which needed this, but if they all do this is fine.