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

new fits_to_tif function, fits_to_jpg moved outside and fits_to_img supports both with new filetype parameter #9

Merged
merged 1 commit into from
Aug 14, 2024

Conversation

LTDakin
Copy link
Contributor

@LTDakin LTDakin commented Aug 14, 2024

Adds a new function fits_to_tif to generate tif files, also since the old function that creates jpgs can be used for all file types I brought out fits_to_jpg as a wrapper function as to not break other projects using this code.

The new util function fits_to_img has a parameter file_type to accommodate for future file types needed.

Also I didn't include the change to using LANCZOS since theres already a PR for it here

Copy link
Contributor

@jnation3406 jnation3406 left a comment

Choose a reason for hiding this comment

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

Looks good

@LTDakin LTDakin merged commit 59147e4 into master Aug 14, 2024
1 check failed
@LTDakin LTDakin deleted the feature/fits-to-tif branch August 14, 2024 21:32
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