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

spectra as 32-bit instead of 64-bit #85

Closed
sbailey opened this issue Oct 29, 2015 · 1 comment · Fixed by #212
Closed

spectra as 32-bit instead of 64-bit #85

sbailey opened this issue Oct 29, 2015 · 1 comment · Fixed by #212

Comments

@sbailey
Copy link
Contributor

sbailey commented Oct 29, 2015

Update io.write_frame() to write 32-bit floats, and update io.read_frame() to cast back to 64-bit double for internal calculations. This is a very simple and easily reproducible form of lossy data compression.
Related: We had a long discussion on the desi-data mailing list about the pros and cons of this vs. FPACK tile compression. FPACK can do somewhat better in terms of compression vs. precision lost, but the basic float vs. double compression seems "good enough" for now.

This is similar to what is already done for io.write_image() and io.read_image().

@sbailey
Copy link
Contributor Author

sbailey commented Feb 28, 2016

Adding to pipeline v0.0 milestone, though this could be optional for that software release.

@sbailey sbailey modified the milestones: Pipeline 2016b, Pipeline 2016a Apr 6, 2016
@sbailey sbailey mentioned this issue Jun 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant