search

twVolc / PyCamPermanent

Permanent PiCam (SO2) installation project software
GNU General Public License v3.0
1 stars 2 forks source link

Menu tidying ahead of real-time-processing work #137

Closed tpering closed 1 month ago

tpering commented 1 month ago

renames some toolbars, moves locations, updated version, removed redundant

twVolc commented 1 month ago

How do you feel about me moving the Calibration tab into the Processing Settings tab as a drop-down list? I think this way we keep all of the processing settings inside a single drop-down menu which makes it easier for a user to follow through incrementally to ensure they've set everything up.

Cross-correlation should also be part of Processing Settings as it has the flow_nadeau setup. I've opened an issue to integrate this frame into the Plume velocity settings frame, but until that has been done I think we should keep that one in Processing Settings - with a changed name from Cross-correlation plot to Cross-correlation to indicate that it has settings within it.

I have the code for this written on this branch, so if you're happy with these changes I can push them.

toolbar

tpering commented 1 month ago

Yep - happy with those changes, nice and logical.

On Wed, Jun 5, 2024 at 11:57 AM twVolc @.***> wrote:

How do you feel about me moving the Calibration tab into the Processing Settings tab as a drop-down list? I think this way we keep all of the processing settings inside a single drop-down menu which makes it easier for a user to follow through incrementally to ensure they've set everything up.

Cross-correlation should also be part of Processing Settings as it has the flow_nadeau setup. I've opened an issue to integrate this frame into the Plume velocity settings frame, but until that has been done I think we should keep that one in Processing Settings - with a changed name from Cross-correlation plot to Cross-correlation to indicate that it has settings within it.

I have the code for this written on this branch, so if you're happy with these changes I can push them.

toolbar.png (view on web) https://github.com/twVolc/PyCamPermanent/assets/19552251/9d3241de-455b-43a6-a738-05848a3d46cf

— Reply to this email directly, view it on GitHub https://github.com/twVolc/PyCamPermanent/pull/137#issuecomment-2149503272, or unsubscribe https://github.com/notifications/unsubscribe-auth/AB36N24Q35P3WRS54TQOP43ZF3VI3AVCNFSM6AAAAABIYK625GVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBZGUYDGMRXGI . You are receiving this because you authored the thread.Message ID: @.***>