bellonemauro / BioZone6

Fluicell software Biozone6
Other
0 stars 1 forks source link

Only one integration mode for the IonOptix Integration #3

Open NiekWe opened 1 year ago

NiekWe commented 1 year ago

:warning: This is a issue tracker :warning:

remove all operational modes except "standard and regular" for the IonOptix integration

Your Environment

Expected Behavior

Only one operational mode left

Current Behavior

4 different operational modes, for UWZ only one operational mode needed.

Possible Solution

image5701

bellonemauro commented 1 year ago

This sounds kind of weird in my perspective. If there is only one OM, then nothing would be needed there so we could keep it clean. The same is in the case of biopen (no operational modes are available there). What do you think if we just remove/hide/gray it out?

NiekWe commented 1 year ago

Greying out the whole thing (like BioPen) sounds good to me.

bellonemauro commented 1 year ago

ok, but it is not solved yet, I will do that and close the issue when the new version will be out

bellonemauro commented 1 year ago

@NiekWe An additional question remains, if I gray out the OM in the interface, what would happen to the commands in the protocols? can one run protocols with OM in ionOptix?

NiekWe commented 1 year ago

I guess your question regards to these commands image Could on SnR_On_button and SnR_Off_button remain available in the IonOPtix Integration?

bellonemauro commented 1 year ago

well, graying out something in the interface using a flag is simple. Changing protocols accordingly would require quite a lot of tweaks and flags around that would make the whole thing messy. This was the reason why originally we decided to split biopen from biozone. The original idea of biozone was just to be -the same- as biopen but with 6 channels, when we got to protocols we realized that the idea was not feasible (not easily at least) and thus the software was split.

One clean solution can be to leave protocols as they are, and just warn the user that those commands cannot be run (if present in the protocol) and prevent their execution.

NiekWe commented 1 year ago

Or not warn them at all, right now all these buttons (SnS, LnS, SnR, LnR) lead to exact the same protocol. If the user takes the wrong button the results would be exactly the same as taking the right button.

bellonemauro commented 1 year ago

This can be done, I will work on this a bit.

NiekWe commented 1 year ago

Could the operational modes be made transparent completely, so you cannot see them?