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

TST: Use a slightly more realistic test case #1

Merged
merged 1 commit into from
Sep 7, 2022

Conversation

pllim
Copy link

@pllim pllim commented Sep 7, 2022

Description

This pull request is to fix the failing test case in CI.

@duytnguyendtn
Copy link
Owner

Was there an issue with the test cube I made? :/

@pllim
Copy link
Author

pllim commented Sep 7, 2022

Not sure, never did find the exact cause. arange with handwritten WCS is probably not recommended anyway. We have premade image cube from conftest.py.

@pllim
Copy link
Author

pllim commented Sep 7, 2022

CI passed. You can ignore the change log check here.

@duytnguyendtn duytnguyendtn merged commit 1af7936 into duytnguyendtn:linesubset Sep 7, 2022
@duytnguyendtn
Copy link
Owner

Thank you so much @pllim !! 🙇

@pllim pllim deleted the pr1583-debug branch September 7, 2022 16:53
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.

2 participants