search

rsaim / supplementary

Portal to analyze and visualize results of DTU students.
1 stars 0 forks source link

Fix downloading/syncing data from dropbox #19

Open tezas opened 4 years ago

tezas commented 4 years ago

https://github.com/rsaim/supplementary/blob/acc538f76e4bd05be84bcb1d8212d406744627f3/src/python/dropbox_updown.py#L72

I tried running the above script as suggested in README apparently that didn't work. Are we expecting dtu_results to be made by user in the data directory present in codebase? Reason behind this: Scripts just trying to download README as it finds nothing else n the local dir. I felt the positional way of providing the cmdline arguments is not informative. I will try to provide it the optional structure just to provide the information from where to where we are syncing the data.Please suggest here.

NOTE: Workaround is to download a zip file (dd5c6ee) containing all the data from dropbox and then use dropbox_updown.py for making updates the dropbox data.

rsaim commented 4 years ago

What's the exception you are facing?

rsaim commented 4 years ago

I tried running the above script as suggested in README apparently that didn't work. Are we expecting dtu_results to be made by user in the data directory present in codebase? Reason behind this: Scripts just trying to download README as it finds nothing else n the local dir.

Try passing --yes flag at the end. The script might be prompting you to confirm whether to create the dirs or not.

I felt the positional way of providing the cmdline arguments is not informative. I will try to provide it the optional structure just to provide the information from where to where we are syncing the data. Please suggest here.

I agree its a bit unintuitive. However, as its a copy from the dropbox github, I thought it is not worth spending time improving the script.

rsaim commented 4 years ago

I have updated README of the project.

himanshuhy commented 4 years ago

+1 Facing the same issue. Ill add details after deep dive, but I think we need to have the data already downloaded for this script to actually work. I will test and get back here.

rsaim commented 4 years ago

+1 Facing the same issue. Ill add details after deep dive, but I think we need to have the data already downloaded for this script to actually work. I will test and get back here.

Are you passing --yes at the end of the command?

himanshuhy commented 4 years ago

+1 Facing the same issue. Ill add details after deep dive, but I think we need to have the data already downloaded for this script to actually work. I will test and get back here.

Are you passing --yes at the end of the command?

Yes

rsaim commented 4 years ago

@himanshuhy or @tezas, let's get on a quick call. I'll try to debug.

rsaim commented 4 years ago

I hope this is not a blocker for us now. @himanshuhy, could you please share the work-around here?

I am repurposing this Issue to fix the issues with dropbox downloading and syncing.

himanshuhy commented 4 years ago

Its there in https://github.com/rsaim/supplementary/commit/dd5c6eebdc4246d3ac37872ec0886da9c9a32ce6

rsaim commented 4 years ago

Thanks @himanshuhy