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

Fix Create constant raster #92

Closed
nmaarnio opened this issue Mar 4, 2024 · 2 comments · Fixed by #102
Closed

Fix Create constant raster #92

nmaarnio opened this issue Mar 4, 2024 · 2 comments · Fixed by #102
Assignees
Labels
bug Something isn't working Processing algorithms Feature/issue related to QGIS processing algorithms

Comments

@nmaarnio
Copy link
Collaborator

nmaarnio commented Mar 4, 2024

Create constant raster does not work currently. Should be matched to the toolkit CLI definition.

@nmaarnio nmaarnio added bug Something isn't working Processing algorithms Feature/issue related to QGIS processing algorithms labels Mar 4, 2024
@lehtonenp lehtonenp self-assigned this Mar 11, 2024
@lehtonenp
Copy link
Collaborator

lehtonenp commented Mar 13, 2024

@nmaarnio, any help with making the target epsg optional and use the project's epsg if it is not chosen? Should I override the processAlgorithm()?

I'm adding on to this comment: Should the CRS be optional in the Plugin?

@nmaarnio
Copy link
Collaborator Author

I think following the logic of QGIS's own Create constant raster layer is a good idea. So CRS is not optional. Overall our algorithm will be quite identical to that one, with the exception that we don't expose number type definition to users.

@lehtonenp lehtonenp linked a pull request Mar 14, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Processing algorithms Feature/issue related to QGIS processing algorithms
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants