search

pangeo-forge / C-iTRACE-feedstock

Pangeo Forge feedstock for C-iTRACE.
https://pangeo-forge.org
Apache License 2.0
0 stars 0 forks source link

Move preprocessor imports into function body #2

Closed cisaacstern closed 2 years ago

cisaacstern commented 2 years ago

Based on recent experience with this issue, e.g. in https://github.com/pangeo-forge/staged-recipes/pull/188#issuecomment-1261563283, it seems that, among any other bugs to fix on this recipe, we will at the minimum need to move these imports inside the function body.

cisaacstern commented 2 years ago

cc @jordanplanders I think we never hit this issue, because the memory issues discussed in https://github.com/pangeo-forge/staged-recipes/pull/176 were killing our recipe runs before we got to this part of the code. But I'll throw this fix in now, because it's almost guaranteed to be necessary after we resolve the (harder to fix) memory issue.