Murali-group / CyGraphSpace

A GraphSpace app for Cytoscape
GNU General Public License v3.0
1 stars 2 forks source link

Move export/update progress into background #14

Closed LHuang2019 closed 6 years ago

LHuang2019 commented 6 years ago

Currently when in progress of exporting or updating a network. The Cytoscape app freezes and user needs to wait for the progress to end to be able to use Cytoscape again.

To avoid this, CyGraphSpace should do the following:

LHuang2019 commented 6 years ago

@adbharadwaj @jlaw9 In order to show progress in Task History. It seems that we need to implement Task Monitor into our code base. Such implementation requires us to refactor our code base so that our entire export and update algorithm is wrapped by a Task class. For me, this refactor requires considerable amount of time (8 hours minimum), but if done correctly, the code base will be much more elegant, fits the Cytoscape app development ideology (OSGi), and it will be easier to add newer functionalities into the app without cluster up the code base.

jlaw9 commented 6 years ago

Updating a network is pretty slow (~30 sec compared to uploading a new graph which is a few seconds), and on cytoscape, it looks like nothing is happening. If you show the currently running tasks, it shows the network is being updated, but I think it would be helpful to show a status bar or an icon showing it's in progress

LHuang2019 commented 6 years ago

The issue of the notification can be solved by using the result panel to display related information. #20