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] Map is not added to the map window after running analysis #4166

Closed
lindakarlovska opened this issue Aug 12, 2024 · 4 comments · Fixed by #4174
Closed

[Bug] Map is not added to the map window after running analysis #4166

lindakarlovska opened this issue Aug 12, 2024 · 4 comments · Fixed by #4174
Labels
bug Something isn't working GUI wxGUI related
Milestone

Comments

@lindakarlovska
Copy link
Contributor

lindakarlovska commented Aug 12, 2024

Describe the bug

Map is not added to the map window after running some tool generating map output.

To reproduce

I have run the r.watershed analysis and the analysis run successfully.
However, after finishing the script flow I got this error in the Console:


Traceback (most recent call last):
  File "/home/linduska/grass/dist.x86_64-pc-linux-
gnu/gui/wxpython/core/gconsole.py", line 830, in OnCmdDone

self.mapCreated.emit(
  File "/home/linduska/grass/dist.x86_64-pc-linux-
gnu/etc/python/grass/pydispatch/signal.py", line 232, in
emit

dispatcher.send(signal=self, *args, **kwargs)
  File "/home/linduska/grass/dist.x86_64-pc-linux-
gnu/etc/python/grass/pydispatch/dispatcher.py", line 340, in
send

response = robustapply.robustApply(
  File "/home/linduska/grass/dist.x86_64-pc-linux-
gnu/etc/python/grass/pydispatch/robustapply.py", line 90, in
robustApply

return receiver(*arguments, **named)
TypeError
:
TaskFrame.OnMapCreated() got an unexpected keyword argument
'add'
-->

Expected behavior

Should add a new map to a map window.

System description

I have not observed it thoroughly, but it seems similar to #4124.
Please @echoix , could you look at it?

@lindakarlovska lindakarlovska added bug Something isn't working GUI wxGUI related labels Aug 12, 2024
@echoix
Copy link
Member

echoix commented Aug 12, 2024

The commit that changed lambdas that was problematic for #4124 has been reverted since.

@veroandreo
Copy link
Contributor

I can confirm this. Trying to run a tool from the GUI, in this case r.neighbors, yields the following in the console, and no map displayed.

Traceback (most recent call last):
  File "/home/vandreo/software/grass-dev/dist.x86_64-pc-
linux-gnu/gui/wxpython/core/gconsole.py", line 830, in
OnCmdDone

self.mapCreated.emit(
  File "/home/vandreo/software/grass-dev/dist.x86_64-pc-
linux-gnu/etc/python/grass/pydispatch/signal.py", line 232,
in emit

dispatcher.send(signal=self, *args, **kwargs)
  File "/home/vandreo/software/grass-dev/dist.x86_64-pc-
linux-gnu/etc/python/grass/pydispatch/dispatcher.py", line
340, in send

response = robustapply.robustApply(

  File "/home/vandreo/software/grass-dev/dist.x86_64-pc-
linux-gnu/etc/python/grass/pydispatch/robustapply.py", line
90, in robustApply

return receiver(*arguments, **named)

TypeError
:
TaskFrame.OnMapCreated() got an unexpected keyword argument
'add'

System info

GRASS version: 8.5.0dev                                                         
Code revision: c53c938bac                                                       
Build date: 2024-08-04                                                          
Build platform: x86_64-pc-linux-gnu                                             
GDAL: 3.7.1                                                                     
PROJ: 9.2.1                                                                     
GEOS: 3.12.0                                                                    
SQLite: 3.42.0                                                                  
Python: 3.11.6                                                                  
wxPython: 4.2.1                                                                 
Platform: Linux-6.5.0-44-generic-x86_64-with-glibc2.38                          

@nilason
Copy link
Contributor

nilason commented Aug 13, 2024

c88fc0f is the culprit.

@nilason
Copy link
Contributor

nilason commented Aug 13, 2024

c88fc0f is the culprit.

(I need not to stress the importance of testing the GUI after similar changes in the future).

petrasovaa added a commit that referenced this issue Aug 14, 2024
Fixes #4166, something must have changed in the pydispatch library, it looks like before it ignored if the handler didn't have all the parameters.

Co-authored-by: Edouard Choinière <27212526+echoix@users.noreply.github.com>
@neteler neteler added this to the 8.5.0 milestone Aug 15, 2024
landam pushed a commit to landam/grass that referenced this issue Aug 22, 2024
Fixes OSGeo#4166, something must have changed in the pydispatch library, it looks like before it ignored if the handler didn't have all the parameters.

Co-authored-by: Edouard Choinière <27212526+echoix@users.noreply.github.com>
Mahesh1998 pushed a commit to Mahesh1998/grass that referenced this issue Sep 19, 2024
Fixes OSGeo#4166, something must have changed in the pydispatch library, it looks like before it ignored if the handler didn't have all the parameters.

Co-authored-by: Edouard Choinière <27212526+echoix@users.noreply.github.com>
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.

5 participants