pepijn-devries / CopernicusMarine

Subset and download marine data from EU Copernicus Marine Service Information. Import data on the oceans physical and biogeochemical state from Copernicus into R without the need of external software.
https://pepijn-devries.github.io/CopernicusMarine/
GNU General Public License v3.0
25 stars 3 forks source link

Locked process causes infinite wait #38

Closed Ge0rges closed 9 months ago

Ge0rges commented 9 months ago

Hello,

This is likely to be a very unclear bug report and I apologize, but I thought it should be raised.

I've been playing around with copernicusmarine to get data back using the python API specifically using copernicusmarine.read_dataframe. This involved quitting the process while running, attaching a debugger, etc.

I also experimented with parallel calls to this function, and then reverted back to sequential only calls.

At some point, running my script would hang on that function for ever (tested over 12 hours). I was not able to use the debugger to step into the function oddly (that's more an issue with me than anything else probably), but when the process quite I could see it was waiting for a thread lock.

The only way this was fixed was by reinstalling copernicusmarine using pip.

Perhaps this points to a bug inside the framework? If this is not a useful report, feel free to close it. I will try to reproduce and update if I find anything. This is in python 3.11.7 using copernicusmarine 1.0.1

pepijn-devries commented 9 months ago

Hi @Ge0rges ,

Note that you have filed your report at the repository of the R package. Whereas the issue that you are describing concerns the Python API (https://pypi.org/project/copernicusmarine/). The R package does not use the Python API. So I'm afraid that I can't help you with this. I suggest you try to contact the developers/maintainers of the Python API. If you have an issue with the R package you can report it here (preferably with a reproducible example).

Pepijn

Ge0rges commented 9 months ago

Apologies for the oversight.