Opened 5 years ago

Closed 5 years ago

Last modified 5 years ago

#3734 closed defect (fixed)

Location wizard on python 3 seems to be broken

Reported by: pmav99 Owned by: grass-dev@…
Priority: normal Milestone: 7.8.0
Component: wxGUI Version: svn-trunk
Keywords: location wizard Cc:
CPU: x86-64 Platform: Linux

Description

I compiled grass gis from source using python 3 and the location wizard seems to be broken. More specifically clicking on the "New" location button shows a window with the text:

Unable to read list: dict_items object has no attribute 'sort'

This is rather typical in python2 code which runs with the python3 interpreter. I checked the source code of grass/gui/wxpython/location_wizard/wizard.py and fixed that but then the New button does nothing.

Any suggestions WRT how I could see the full traceback?

BTW, from now on, is python code supposed to be both Python 2 and 3 compatible? Or Python 3 only?

svn diff
Index: location_wizard/wizard.py
===================================================================
--- location_wizard/wizard.py	(revision 73967)
+++ location_wizard/wizard.py	(working copy)
@@ -654,7 +654,8 @@
             self.sourceData = data
 
         try:
-            data.sort()
+            data = sorted(data)
+            #data.sort()
             self.DeleteAllItems()
             row = 0
             for value in data:

Change History (6)

comment:1 by martinl, 5 years ago

Keywords: location wizard added
Milestone: 7.8.0

comment:2 by annakrat, 5 years ago

Please try r73972.

It should be compatible with Python 2.7 and 3.

in reply to:  description comment:3 by martinl, 5 years ago

Replying to pmav99:

BTW, from now on, is python code supposed to be both Python 2 and 3 compatible? Or Python 3 only?

GRASS 7.8 should be Python 2 and 3 compatible. Python3-only release will be probably GRASS 8.0.

in reply to:  2 ; comment:4 by pmav99, 5 years ago

Replying to annakrat:

Please try r73972.

I confirm that this is fixed.

BTw, any pointers WRT how could I debug this on my own?

comment:5 by pmav99, 5 years ago

Resolution: fixed
Status: newclosed

in reply to:  4 comment:6 by annakrat, 5 years ago

Replying to pmav99:

Replying to annakrat:

Please try r73972.

I confirm that this is fixed.

BTw, any pointers WRT how could I debug this on my own?

I ran location wizard from layer manager settings menu and I got the errors in gui console.

Note: See TracTickets for help on using tickets.