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

Address issues raised in #260 #263

Closed
derobins opened this issue Feb 5, 2023 · 1 comment
Closed

Address issues raised in #260 #263

derobins opened this issue Feb 5, 2023 · 1 comment
Assignees
Labels
Component - C Library Core C library issues Component - netCDF netCDF interface and nc* command-line tools Component - Testing Test code, GitHub workflows Component - Tools Command-line tools like dumper and hdiff Priority - 2. Medium ⏹ It would be nice to have this in the next release Type - Improvement Improvements that don't add a new feature or functionality

Comments

@derobins
Copy link
Member

derobins commented Feb 5, 2023

Code review in PR #260 raised some concerns about memory usage (among other things). We should go back over those comments and address code smells.

@derobins derobins added Component - C Library Core C library issues Component - Tools Command-line tools like dumper and hdiff Component - netCDF netCDF interface and nc* command-line tools Component - XDR Component - Testing Test code, GitHub workflows Type - Improvement Improvements that don't add a new feature or functionality Priority - 2. Medium ⏹ It would be nice to have this in the next release labels May 3, 2023
@derobins
Copy link
Member Author

derobins commented May 8, 2023

Most of the issues here are addressed. Those that remain will be cleaned as we refactor and tidy.

@derobins derobins closed this as completed May 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component - C Library Core C library issues Component - netCDF netCDF interface and nc* command-line tools Component - Testing Test code, GitHub workflows Component - Tools Command-line tools like dumper and hdiff Priority - 2. Medium ⏹ It would be nice to have this in the next release Type - Improvement Improvements that don't add a new feature or functionality
Projects
Status: Done
Development

No branches or pull requests

2 participants