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

Adding nc_def_var_quantize()/nc_inq_var_quantize() #2081

Closed
wants to merge 13 commits into from

Conversation

edwardhartnett
Copy link
Contributor

Adding nc_def_var_quantize()/nc_inq_var_quantize() as discussed in recent meeting.

This is a work in progress but I want to start running it through the CI to see if I am breaking anything. ;-)

Fixes #1548

@DennisHeimbigner
Copy link
Collaborator

I eventually came to the conclusion that the nc_inq_var_all function was a mistake
because it gets too cumbersome as parameters are added (like here).
We should consider adding an inq_var_quantize function to the dispatch table
instead of extending inq_var_all.

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.

Provide a way to do bit grooming before compression
2 participants