Skip to content
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.

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

wxGUI/iclass: fix zoom to scatter plot extend #3184

Merged

Conversation

tmszi
Copy link
Member

@tmszi tmszi commented Sep 23, 2023

Describe the bug
Activating Supervised Classification Tool window scatter plots pane zoom to scatter plot extend menu item prints an error message.

To Reproduce
Steps to reproduce the behavior:

  1. Launch Supervised Classification Tool wxGUI g.gui.iclass
  2. On the left Plots pane combobox widget choose Scatter plots item
  3. On the Scatter plots toolbar activate Add scatter plot tool
  4. On the Select imagery group dialog from the name of imagery group listbox widget items choose lsat7_2000
  5. Hit Create/edit group... button widget
  6. On the Create or edit image group dialog, type into Select existing subgroup or enter name of new subgroup combobox widget new subgroup name e.g. test and select some maps from the list to be part of the new cretaed subgroup test (lsat7_2000_10, lsat7_2000_20, lsat7_2000_30, lsat7_2000_40) and hit OK button
  7. Hit OK button on the Create or edit image group dialog
  8. On the next Add scatter plots dialog choose from the x axis combobox widget items lsat7_2000_10 raster map, and from the y axis combobox widget items choose lsat7_2000_20 raster map and hit Add button
  9. Repeat point 9. but with another maps, from the x axis combobox widget items choose lsat7_2000_30 raster map, and from the y axis combobox widget items choose lsat7_2000_40 raster map and hit Add button
  10. On the Add scatter plots dialog hit OK button
  11. Try right mouse click inside scatter plot pane to invoke menu and choose Zoom to scatter plot extend item
  12. See error
Traceback (most recent call last):
  File "/usr/lib/python3.10/site-packages/wx/core.py", line 3427, in <lambda>
    lambda event: event.callable(*event.args, **event.kw) )
  File "/usr/lib64/grass84/gui/wxpython/iscatt/plots.py", line 621, in ShowMenu
    self.plot.ReleaseMouse()
wx._core.wxAssertionError: C++ assertion ""!wxMouseCapture::stack.empty()"" failed at /var/tmp/portage/x11-libs/wxGTK-3.2.2.1-r2/work/wxWidgets-3.2.2.1/src/common/wincmn.cpp(3425) in ReleaseMouse(): Releasing mouse capture but capture stack empty?

Expected behavior
Activating Supervised Classification Tool window scatter plots pane zoom to scatter plot extend menu item should not prints an error message.

System description:

  • Operating System: all
  • GRASS GIS version: all
GRASS nc_spm_08_grass7/landsat:~ > python3 -c "import sys, wx; print(sys.version); print(wx.version())"
3.10.13 (main, Sep 16 2023, 22:24:59) [GCC 12.3.1 20230526]
4.2.0 gtk3 (phoenix) wxWidgets 3.2.2.1

@tmszi tmszi added bug Something isn't working GUI wxGUI related backport to 8.3 PR needs to be backported to release branch 8.3 backport to 8.2 PR needs to be backported to release branch 8.2 labels Sep 23, 2023
@tmszi tmszi added this to the 8.3.1 milestone Sep 23, 2023
@tmszi tmszi merged commit d8f94cf into OSGeo:main Sep 26, 2023
18 checks passed
tmszi added a commit to tmszi/grass that referenced this pull request Sep 26, 2023
tmszi added a commit to tmszi/grass that referenced this pull request Sep 26, 2023
@tmszi tmszi removed backport to 8.3 PR needs to be backported to release branch 8.3 backport to 8.2 PR needs to be backported to release branch 8.2 labels Sep 26, 2023
@tmszi tmszi deleted the wxgui-iclass-fix-zoom_to_scatter_plot_extend branch September 26, 2023 11:53
landam pushed a commit to landam/grass that referenced this pull request Oct 25, 2023
neteler pushed a commit to nilason/grass that referenced this pull request Nov 7, 2023
HuidaeCho pushed a commit to HuidaeCho/grass that referenced this pull request Jan 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working GUI wxGUI related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants