DistributedScience / Distributed-CellProfiler

Run encapsulated docker containers with CellProfiler in the Amazon Web Services infrastructure.
https://distributedscience.github.io/Distributed-CellProfiler/
Other
37 stars 24 forks source link

DOWNLOAD_FILES doesn't fully bypass S3FS #132

Closed ErinWeisbart closed 1 year ago

ErinWeisbart commented 2 years ago

If DOWNLOAD_FILES is True in the config, it looks to the load_data_csv for which files to download. However, it's looking at os.path.join(DATA_ROOT,message['data_file']) and DATA_ROOT = '/home/ubuntu/bucket' so S3FS is still required to mount the bucket correctly before downloading the files locally.

It seems we change this behavior.

ErinWeisbart commented 1 year ago

Fixed in #130.