snu-quiqcl / qiwis

QuIqcl Widget Integration Software
MIT License
6 stars 2 forks source link

[FEATURE] Update a part adding/removing DBs #82

Closed BECATRUE closed 1 year ago

BECATRUE commented 1 year ago

image

In the above logging messages, pay attention to messages of which timestamp is 16:50:49.

There are many duplicated messages, such as "Polled database is not selected.". I just removed a dataset 'A.sqlite' in dbmgr, but it occured duplicated messages.

This occurs because each update calls the function below.

self.viewerFrame.dbBox.clear() self.viewerFrame.dbBox.addItem("")

Other than this, there are several problems such that we always replace all database list to received list, and it is so inefficient.

Therefore, I want to implement as below:

Originally posted by @BECATRUE in https://github.com/snu-quiqcl/swift/issues/81#issuecomment-1467311437

kangz12345 commented 1 year ago

Please provide more information about the bug. What is the problem and how are you going to fix it?

BECATRUE commented 1 year ago

Please provide more information about the bug. What is the problem and how are you going to fix it?

I updated the explanations.

BECATRUE commented 1 year ago

I updated updateDB() in poller.

Please check it! @kangz12345

kangz12345 commented 1 year ago

I updated updateDB() in poller.

Please check it! @kangz12345

Do you mean in the commit eda90e2 in branch BECATRUE/82/update_db?

BECATRUE commented 1 year ago

I updated updateDB() in poller. Please check it! @kangz12345

Do you mean in the commit eda90e2 in branch BECATRUE/82/update_db?

Yes, it is right!