GETs remote file twice

[yarikoptic]

visiting http://nwbexplorer.opensourcebrain.org/nwbfile=https://datasets-dev.datalad.org/dandi/dandisets/000027/sub-RAT123/sub-RAT123.nwb and looking at apache logs, shows

XXX - - [05/Aug/2021:17:05:10 -0400] "GET /dandi/dandisets/000027/sub-RAT123/sub-RAT123.nwb HTTP/1.1" 200 18792
XXX - - [05/Aug/2021:17:05:18 -0400] "GET /dandi/dandisets/000027/sub-RAT123/sub-RAT123.nwb HTTP/1.1" 200 18792

with I guess first one to display details, and then 2nd one appears shortly after a spinner in Python tab stops spinning so - for that Python session? ... For a more sizeable http://nwbexplorer.opensourcebrain.org/nwbfile=https://datasets-dev.datalad.org/dandi/dandisets/000005/sub-anm184389/sub-anm184389_ses-20130207_behavior+ecephys.nwb - the same effect of two full 286MB download requests.

XXX - - [05/Aug/2021:17:10:55 -0400] "GET /dandi/dandisets/000005/sub-anm184389/sub-anm184389_ses-20130207_behavior+ecephys.nwb HTTP/1.1" 200 299000608
XXX - - [05/Aug/2021:17:11:28 -0400] "GET /dandi/dandisets/000005/sub-anm184389/sub-anm184389_ses-20130207_behavior+ecephys.nwb HTTP/1.1" 200 299000608

It seems that ideally it should be avoided, and a singular GET should be sufficient.

[filippomc]

Hi @yarikoptic thanks for the detailed analysis. Yes this is certainly related to the python console file loading.

A strategy can be to make a local copy of the downloaded file and use that copy for the Python console.
Need to ensure that the Python console load starts after the dowload is complete

[filippomc]

Relevant code parts:

Handle action file loading action from the frontend :

nwb-explorer/webapp/redux/middleware/nwbMiddleware.js

Line 117 in c7e45cd

nwbFileService.loadNWBFileInNotebook(store.getState().nwbfile.nwbFileUrl).then(

Trigger notebook load action:

nwb-explorer/webapp/redux/middleware/nwbMiddleware.js

Line 133 in c7e45cd

store.dispatch(loadNWBFileInNotebook);

The trigger must be changed, here we are not sure that the download was complete.

Backend entry point:

nwb-explorer/nwb_explorer/nwb_main.py

Line 46 in c7e45cd

geppetto.set_nwb_file(nwbfilename)