kitzeslab / opensoundscape

Open source, scalable software for the analysis of bioacoustic recordings
http://opensoundscape.org
MIT License
137 stars 16 forks source link

ResourceWarning: Implicitly cleaning up <TemporaryDirectory '/tmp/tmp1k41qlgewandb'> #775

Closed louisfh closed 4 weeks ago

louisfh commented 1 year ago

I keep seeing this Warning. This happens even in scripts where I am not training or using a model, so I don't understand why temp files that appear to be related to wandb are being created.

Here's the full warnings:

/home/lfh/miniconda3/envs/opso_091_pip_py39/lib/python3.9/tempfile.py:821: ResourceWarning: Implicitly cleaning up <TemporaryDirectory '/tmp/tmp1k41qlgewandb'>
  _warnings.warn(warn_message, ResourceWarning)
/home/lfh/miniconda3/envs/opso_091_pip_py39/lib/python3.9/tempfile.py:821: ResourceWarning: Implicitly cleaning up <TemporaryDirectory '/tmp/tmp0wolavyawandb-artifacts'>
  _warnings.warn(warn_message, ResourceWarning)
/home/lfh/miniconda3/envs/opso_091_pip_py39/lib/python3.9/tempfile.py:821: ResourceWarning: Implicitly cleaning up <TemporaryDirectory '/tmp/tmpv7paddovwandb-media'>
  _warnings.warn(warn_message, ResourceWarning)
/home/lfh/miniconda3/envs/opso_091_pip_py39/lib/python3.9/tempfile.py:821: ResourceWarning: Implicitly cleaning up <TemporaryDirectory '/tmp/tmpghepu6pzwandb-media'>
  _warnings.warn(warn_message, ResourceWarning)
sammlapp commented 1 year ago

this seems related https://github.com/wandb/wandb/issues/5258

sammlapp commented 9 months ago

the issue hasn't been resolved but there's now a workaround https://github.com/wandb/wandb/issues/5258#issuecomment-1897623369 and recent activity

sammlapp commented 9 months ago

Ah the issue is now closed as resolved, an updated wandb dependency should fix this (as soon as there’s a new wandb release)

sammlapp commented 6 months ago

@louisfh can you confirm that this is fixed if we update the wandb dependency?

louisfh commented 2 months ago

I'm no longer able to reproduce this behavior with wandb 0.17.7 (version in poetry lock file currently). I assume this has been fixed. Closing the issue and we can reopen if it reappears.

sammlapp commented 1 month ago

Resolved in wandb 0.16.3 https://github.com/wandb/wandb/milestone/53 and we now require >=0.17.0, so this will be resolved with next merge of develop to master