Opened 9 years ago

Closed 9 years ago

Last modified 9 years ago

#1466 closed defect (fixed)

QGIS Processing needs patch to work with GRASS7

Reported by: kalxas Owned by: live-demo@…
Priority: critical Milestone: OSGeoLive9.0
Component: OSGeoLive Keywords:
Cc: live-demo@…, neteler

Description

https://github.com/kalxas/QGIS/commit/07b98ad76c85939996dd67e23b2c218cba6ee04d

Thanks Markus for the patch, which is now merged in QGIS git master branch. I rebased it to the QGIS 2.6 git branch, so we need to apply this patch now.

Attachments (1)

grass7_output_error_RC1.png (357.5 KB ) - added by darkblueb 9 years ago.

Download all attachments as: .zip

Change history (11)

comment:1 by kalxas, 9 years ago

Patch committed on [12303]

comment:2 by kalxas, 9 years ago

Resolution: fixed
Status: newclosed

The patch seems to work fine in latest nightly build.

comment:3 by darkblueb, 9 years ago

Resolution: fixed
Status: closedreopened

sorry to say - this is not fixed.. and, I have been slow to retest so this msg is late. I have RC1 open .. I am using QGis .. Processing Interface set to 'advanced' ..

the first time, I looked in QGis preferences and another settings window, and did not see how to enable GRASS7. So it is not always obvious to go to the Processing providers interface to enable GRASS7 and disable GRASS.

Next, SAGA is not found and there appears to be no way to enable it.

Next, after GRASS7 is enabled, no result layer is showing. I see that the Processing output directory is set to /tmp/processing

Looking in /tmp/processing/job_uuid there is not output.. for example.. see screenshot..

user@live85-rc1:/tmp/processing$ ls -lh
...
drwxrwxr-x 2 user user 4.0K Feb 22 16:04 16c68e74cec84a7ba62994e068c29abb/
...
user@live85-rc1:/tmp/processing$ ls -lh 16c68e74cec84a7ba62994e068c29abb/
total 0

by darkblueb, 9 years ago

Attachment: grass7_output_error_RC1.png added

comment:4 by neteler, 9 years ago

Cc: neteler added

comment:5 by neteler, 9 years ago

I have made a small addition in r12407 in order to note the switch to "advanced Interface" in Processing. However, when opening zipcodes_wake.shp in QGIS and performing a dissolve on "name", i.e. running v.dissolve from GRASS GIS 7, it segfaults somewhere.

comment:6 by kalxas, 9 years ago

Milestone: OSGeoLive8.5OSGeoLive9.0

I believe we need to apply this commit for processing to work with grass7 in 9.0 release:

https://github.com/qgis/QGIS/commit/2a14ffd281d0a0e99a0a899622a29ca0efdb0852

comment:7 by kalxas, 9 years ago

Testing on the latest build (146):

I was able to enable GRASS7 support from Processing->Options->Providers->GRASS7->Activate

vector.clean worked fine, same as raster.grow

Any volunteer from the QGIS, GRASS teams to test deeper?

comment:8 by kalxas, 9 years ago

Testing on build 150: SAGA provider also works fine within QGIS processing.

comment:9 by kalxas, 9 years ago

Resolution: fixed
Status: reopenedclosed

in reply to:  7 comment:10 by neteler, 9 years ago

Replying to kalxas:

Testing on the latest build (146):

I was able to enable GRASS7 support from Processing->Options->Providers->GRASS7->Activate

vector.clean worked fine, same as raster.grow

Excellent.

Any volunteer from the QGIS, GRASS teams to test deeper?

This would best be asked on grass-user or grass-dev.

Note: See TracTickets for help on using tickets.