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

Create ImageReader subclasses #53

Merged
merged 2 commits into from
Oct 30, 2023
Merged

Create ImageReader subclasses #53

merged 2 commits into from
Oct 30, 2023

Conversation

jeremykubica
Copy link

Create specialized subclasses of ImageReader so that we do not need to pass in functions to read the data (they are instead part of the subclass).

Add tests for the HSCImageReader and DC2ImageReader.

Copy link
Owner

@grantmerz grantmerz left a comment

Choose a reason for hiding this comment

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

This is a nice way to do it. Maybe something similar for the key_mapper functions would be useful?

@jeremykubica jeremykubica merged commit bedefa9 into main Oct 30, 2023
1 of 3 checks passed
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