ibllib still uses functions that overlap with those in one.remote.globus. Currently it isn't particularly easy to transfer data using the Globus class in ONE, and the CI download_data script also contain redundant code. To unify all this and to make it more user-friendly...
[x] Add logout method to Globus class
[x] Separate refresh token from setup routine?
[x] Remove ibllib.io.globus in favour of one.remote.globus (see branch globusDeprecation)
[x] Move login methods to one.remote.globus
[x] Add example documentation
[ ] Refactor ci.download_data to use Globus class, convert from script to function
[ ] Improve logging (copy code from ci.download_data)
[x] Test for Globus.download_file
[x] Globus.download_file should return list of file paths
k1o0
commented
1 year ago
ibllib still uses functions that overlap with those in one.remote.globus. Currently it isn't particularly easy to transfer data using the Globus class in ONE, and the CI download_data script also contain redundant code. To unify all this and to make it more user-friendly...