OSOceanAcoustics / echopype

Enabling interoperability and scalability in ocean sonar data analysis
https://echopype.readthedocs.io/
Apache License 2.0
99 stars 73 forks source link

Incorrect Path for Temporary File Generation in `check_file_permissions` Method #1098

Open Sohambutala opened 1 year ago

Sohambutala commented 1 year ago

The check_file_permissions method in the codebase is used to verify file permissions by generating a temporary file in the target path. However, it has been observed that this method is incorrectly creating the temporary file in the parent directory (one level up) of the target path instead of the intended location.

image

File Directory Root : /Users/soham/Desktop/EchoWorkSpace/echoflow/notebooks/combined_files Test file generated at : /Users/soham/Desktop/EchoWorkSpace/echoflow/notebooks/.f0605b17-da68-45e8-95b4-f40281cdca22

lsetiawan commented 1 year ago

Thanks for catching this bug @Sohambutala!