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

refactor: simplify image format dtype validation #1046

Merged
merged 3 commits into from
Dec 20, 2024

Conversation

pratapvardhan
Copy link
Contributor

@pratapvardhan pratapvardhan commented Dec 7, 2024

This PR consolidates dtype validation logic by using a lookup. Keeps the existing functionality intact, probably reduces repetitive code and makes it maintainable?

note: src/titiler/core tests are passing.

cc @vincentsarago – that said, if you feel this change isn’t really necessary, feel free to reject the PR. :)

@vincentsarago
Copy link
Member

@pratapvardhan pratapvardhan force-pushed the core-ref branch 2 times, most recently from 2d3509d to 60c88a5 Compare December 13, 2024 12:45
@pratapvardhan
Copy link
Contributor Author

@vincentsarago resolved merge conflict with changes.md, this should be good for review now.

@vincentsarago vincentsarago merged commit 1846bde into developmentseed:main Dec 20, 2024
10 checks passed
@pratapvardhan pratapvardhan deleted the core-ref branch December 20, 2024 09:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants