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

"Contour" returns error if output is temporary (wrong parameter passed?) #54952

Closed
2 tasks done
gioman opened this issue Oct 17, 2023 · 17 comments
Closed
2 tasks done

"Contour" returns error if output is temporary (wrong parameter passed?) #54952

gioman opened this issue Oct 17, 2023 · 17 comments
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Feedback Waiting on the submitter for answers Processing Relating to QGIS Processing framework or individual Processing algorithms

Comments

@gioman
Copy link
Contributor

gioman commented Oct 17, 2023

What is the bug or the crash?

The "contour" tool in the Processing toolbox seems to have lost the ability to output to temp file. If specifying an real output (i.e. gpkg or shp) it works as expected. The problem seems to be to be the parameter -f "Interlis 2" that it passed, and probably should not:

GDAL command:
gdal_contour -b 1 -a ELEV -i 200.0 -f "Interlis 2" GPKG:C:/Users/giovanni/Desktop/SVR/test.gpkg:mdt C:/Users/giovanni/AppData/Local/Temp/processing_GXQWCj/cdcbd9ff07604c918419db19f781250c/OUTPUT.xtf
GDAL command output:
Warning 1: Model file not specified.
Process returned error code 1
Execution completed in 0.36 seconds
Results:
{'OUTPUT': 'C:/Users/giovanni/AppData/Local/Temp/processing_GXQWCj/cdcbd9ff07604c918419db19f781250c/OUTPUT.xtf'}

Loading resulting layers
The following layers were not correctly generated.
• C:/Users/giovanni/AppData/Local/Temp/processing_GXQWCj/cdcbd9ff07604c918419db19f781250c/OUTPUT.xtf
You can check the 'Log Messages Panel' in QGIS main window to find more information about the execution of the algorithm.

Steps to reproduce the issue

Add a DEM to a project and try extract the contour with a temp output.

Versions

3.28.11
3.32.3

Windows 11.

Supported QGIS version

  • I'm running a supported QGIS version according to the roadmap.

New profile

Additional context

No response

@gioman gioman added Processing Relating to QGIS Processing framework or individual Processing algorithms Bug Either a bug report, or a bug fix. Let's hope for the latter! labels Oct 17, 2023
@alexbruy
Copy link
Contributor

-f "Interlis 2" is passed because you seem have "Interlis 2" set as a default output format.

@alexbruy alexbruy added the Feedback Waiting on the submitter for answers label Oct 17, 2023
@gioman gioman closed this as completed Oct 17, 2023
@gioman
Copy link
Contributor Author

gioman commented Oct 17, 2023

@alexbruy that was the case, anyway is very strange I never changed that option and in fact this was also observed on other people machines where QGIS was only recently installed.

@agiudiceandrea
Copy link
Contributor

It seems a recurring issue. I've seen various users reports on forums and facebook groups about the issue due to that setting they actually didn't set.

@agiudiceandrea
Copy link
Contributor

Another issue report about the xtf "Interlis 2" default output vector format: #55110.

@agiudiceandrea
Copy link
Contributor

Another issue report about the nc default output raster format: https://lists.osgeo.org/pipermail/qgis-user/2024-January/053885.html

@agiudiceandrea
Copy link
Contributor

agiudiceandrea commented Feb 17, 2024

@kannes
Copy link
Contributor

kannes commented Feb 26, 2024

xtf and nc are the last entries in the lists for Processing's "Default output [raster|vector] layer extension".

In those buggy profiles where the xtf and nc are erroneously used, the QGIS3.ini contains:

Configuration\DefaultOutputRasterLayerExt=-1
Configuration\DefaultOutputVectorLayerExt=-1

There is some code catching that and use the wanted defaults (gpkg and tif) but I guess there is a bug with the population of the combobox of chooseble formats in the Processing options.

@agiudiceandrea
Copy link
Contributor

Thanks @kannes for looking at this. I also suggested the same root cause in a recent message to the qgis-developer mailing list https://lists.osgeo.org/pipermail/qgis-developer/2024-February/066539.html.
See also #53458.

@agiudiceandrea
Copy link
Contributor

agiudiceandrea commented Feb 28, 2024

@ar-siddiqui
Copy link

A lot of people are reporting this issue for different plugins, they do not remember explicitly changing the settings. Has there been a solution proposed?

@agiudiceandrea
Copy link
Contributor

@ar-siddiqui, if you have experienced the issue about the settings change without your consent, please open an issue report or inform the qgis-developer mailing list replaying to the at thread https://lists.osgeo.org/pipermail/qgis-developer/2024-February/066539.html. If other user have experienced such issue, please ask them to open an issue report.

@agiudiceandrea
Copy link
Contributor

agiudiceandrea commented Apr 25, 2024

Other issue reports: #54375, #55110 (comment), #57250, #57267.

@ar-siddiqui
Copy link

I ran into this issue today as well. I did not change any settings or installed any plugins that might have updated the settings.

@ar-siddiqui
Copy link

Why has this issue been closed, please reopen this as many people are observing and reporting this. @agiudiceandrea

@agiudiceandrea
Copy link
Contributor

@ar-siddiqui, this issue report has been closed by the reporter, since this specific issue has been fixed by himself just reverting the default output vector extension to the default value.
You can open a new issue report about the unexpected setting change.

@ar-siddiqui
Copy link

A master issue has been created to track this bug #57676

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Either a bug report, or a bug fix. Let's hope for the latter! Feedback Waiting on the submitter for answers Processing Relating to QGIS Processing framework or individual Processing algorithms
Projects
None yet
Development

No branches or pull requests

5 participants