search

dondi / GRNsight

Web app and service for modeling and visualizing gene regulatory networks.
http://dondi.github.io/GRNsight
BSD 3-Clause "New" or "Revised" License
17 stars 8 forks source link

Export to excel not working for Additional Sheets #919

Closed Onariaginosa closed 3 years ago

Onariaginosa commented 3 years ago

Export to excel is not working for optimization_parameters, production_rates, degradation_rates, threshold_b sheets

Onariaginosa commented 3 years ago

I fixed the optimization_parameters export. But now the export functionality in my local machine is very suspicious. Sometimes it exports properly, but other times it redirects me to the site attached: Screenshot from 2021-03-17 13-25-57 And throws this error in my console: Screenshot from 2021-03-17 13-13-07 But when I temporarily changed the code to what they requested, it doesn't export anything or show any signs of exporting.

dondi commented 3 years ago

The 400 status code suggests that our code is returning it—typically a server-level issue is a 5xx status code. So we can try search the sequence for /export-to-excel for cases when a 400 is returned.

Second possibility: keep the dev tools open while trying this and look at the payload just in case there is some information that may be useful.

Onariaginosa commented 3 years ago

It turns out that the issue was in the demo objects. They were structured slightly differently than workbooks that we parse ourselves. Additional sheets usually have a section for data, errors, and warnings, but the demos didn't have that wrapper. They just contained the data directly.

I fixed the optimization_parameters sheet export, as well as the two column sheet exports. Just need to add optimization diagnostics, and export functionality will be fully supported!

dondi commented 3 years ago

Looks like we are on our way here, just waiting for those remaining sheets mentioned above.

Onariaginosa commented 3 years ago

Completed export to excel for all additional sheets! Exported sheets re-upload with no warnings/errors! See pull request #920

dondi commented 3 years ago

Verified by @ahmad00m as fulfilled.