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

[Bug] Graphical Modeler: reading coordinates from map window does not work #3470

Closed
landam opened this issue Mar 2, 2024 · 1 comment · Fixed by #3593
Closed

[Bug] Graphical Modeler: reading coordinates from map window does not work #3470

landam opened this issue Mar 2, 2024 · 1 comment · Fixed by #3593
Assignees
Labels
bug Something isn't working GUI wxGUI related
Milestone

Comments

@landam
Copy link
Member

landam commented Mar 2, 2024

Describe the bug
Reading coordinates from map window fails for tools like r.what called from Graphical Modeler.

To Reproduce
Steps to reproduce the behavior:

  1. Open Graphical Modeler from GUI
  2. Add r.what to the model
  3. Go to Query tab
  4. By 'arrow' icon try to read coordinates from map window

Fails with

Traceback (most recent call last):
  File "/home/martin/src/grass/dist.x86_64-pc-linux-
gnu/gui/wxpython/gui_core/gselect.py", line 2847, in
_onClick

self.mapWin = self._giface.GetMapWindow()
AttributeError: 'GraphicalModelerGrassInterface' object has no attribute
'GetMapWindow'

Expected behavior
Coordinates should be read from map window and stored to coordinates option.

Screenshot

image

System description (please complete the following information):

  • Operating System: Linux
  • GRASS GIS version: 8.4.0dev
@landam landam added bug Something isn't working GUI wxGUI related labels Mar 2, 2024
@landam landam added this to the 8.4.0 milestone Mar 2, 2024
@landam
Copy link
Member Author

landam commented Mar 2, 2024

Originally reported in #3003 (review)

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 a pull request may close this issue.

2 participants