NOAA-GSL / pygraf

ADB's Python graphics package
MIT License
10 stars 11 forks source link

Memory leak fixes #119

Closed christinaholtNOAA closed 2 years ago

christinaholtNOAA commented 2 years ago

This set of changes fixes a memory leak issue we'd been seeing that does not allow running subdomains on smaller Jet partitions (sjet/vjet and tjet/ujet). Summary of major changes:

These changes have been tested for HRRR and RRFS CONUS on a variety of jet partitions with desirable results, most notably the completion of subdomains, even several subdomains.

christinaholtNOAA commented 2 years ago

Excellent suggestion, @cshartsough. I will add a card to remind us to do that.

christinaholtNOAA commented 2 years ago

The linter checks for unused imports, so we should be covered there.

Brian-Jamison commented 2 years ago

Whoops, left my previous comment and just 'approved', sorry for the repeat.

Thanks for the clarification on the linter!