Skip to content

[feat] METEOR overlay cryofeature: clarify the current posture in "recalculate position" message box #3379

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pieleric merged 1 commit into from
Mar 6, 2026
Merged

[feat] METEOR overlay cryofeature: clarify the current posture in "recalculate position" message box #3379

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 5 additions & 2 deletions src/odemis/gui/comp/overlay/cryo_feature.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@
from odemis.gui.comp.overlay.base import DragMixin, WorldOverlay
from odemis.gui.comp.overlay.stage_point_select import StagePointSelectOverlay
from odemis.gui.model import TOOL_FEATURE, TOOL_NONE, TOOL_FIDUCIAL, TOOL_REGION_OF_INTEREST, TOOL_SURFACE_FIDUCIAL
from odemis.acq.move import MicroscopePostureManager, SEM_IMAGING, FM_IMAGING, UNKNOWN
from odemis.acq.move import MicroscopePostureManager, POSITION_NAMES, SEM_IMAGING, FM_IMAGING, UNKNOWN


MODE_EDIT_FEATURES = 1
Expand Down Expand Up @@ -250,9 +250,12 @@ def _update_other_postures(self):
# only possible to set the position in FM, so it's never needed to ask the user.
# TODO: only ask if the posture had been set explicitly in a different posture previously.
if len(self.pm.postures) > 2:
current_posture_name = POSITION_NAMES.get(self.pm.current_posture.value, "current posture")
box = wx.MessageDialog(wx.GetApp().main_frame,
message="Do you want to recalculate this feature position for all other postures?",
message=f"Do you want to update this feature position only for {current_posture_name} "
"or recalculate for all other postures?",
Comment thread
pieleric marked this conversation as resolved.
caption="Recalculate feature positions?", style=wx.YES_NO | wx.ICON_QUESTION | wx.CENTER)
box.SetYesNoLabels("All", f"Only {current_posture_name}")
Comment thread
pieleric marked this conversation as resolved.

ans = box.ShowModal() # Waits for the window to be closed
if ans != wx.ID_YES:
Expand Down