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 gdalmerge nodata #4917

Closed
wants to merge 1,139 commits into from
Closed

Fix gdalmerge nodata #4917

wants to merge 1,139 commits into from

Conversation

gioman
Copy link
Contributor

@gioman gioman commented Jul 24, 2017

Description

Include a few sentences describing the overall goals for this PR (pull request). If applicable also add screenshots.

Checklist

Reviewing is a process done by project maintainers, mostly on a volunteer basis. We try to keep the overhead as small as possible and appreciate if you help us to do so by completing the following items. Feel free to ask in a comment if you have troubles with any of them.

  • Commit messages are descriptive and explain the rationale for changes
  • Commits which fix bugs include fixes #11111 in the commit message next to the description
  • Commits which add new features are tagged with [FEATURE] in the commit message
  • Commits which change the UI or existing user workflows are tagged with [needs-docs] in the commit message and containt sufficient information in the commit message to be documented
  • I have read the QGIS Coding Standards and this PR complies with them
  • This PR passes all existing unit tests (test results will be reported by travis-ci after opening this PR)
  • New unit tests have been added for core changes
  • I have run the scripts/prepare-commit.sh script before each commit

alexbruy and others added 30 commits May 10, 2017 12:28
…ted-rows-fix

Attributetable duplicated rows fix for 2.18
[attributetable] Fix "Move selection to top": for 2.18 regression fixes qgis#15803
If a transform exception occurred while rendering a symbol then
the QgsSymbolRenderContext cleanup code was never called,
leading to a double delete and crash.

Fixes qgis#16377, qgis#15345, and numerous other crashes seen "in the wild"

Possibly refs qgis#16385

(cherry-picked from fefa572)
Fix crash when transform errors occur while rendering
…6492-backport

[bugfix] Attribute table crashes QGIS when deleting selected
We do not need a new evaluated value all the time

Fix qgis#16545
$ scripts/update_ts_files.sh -b $builddir -f de
$ linguist i18n/qgis_de.ts
$ scripts/update_ts_files.sh -b $builddir de
$ git add doc/TRANSLATORS i18n/qgis_de.ts
$ git commit
nyalldawson and others added 28 commits July 14, 2017 11:02
Line pattern fill was not calling stopRender on the line symbol!

(cherry-picked from 6067d51)
The default comparison code for two QVariants does a really crappy job
when it comes to NULL values.
I.e. it will return true for `QVariant( 0 ) == QVariant( QVariant::Int )`
We work around this issue here by returning an invalid QVariant instead,
which - and luckily also Qt is aware of this - also is different from 0.

Fix qgis#16745
proper fix in master bae6d56 could not be cherry-picked easily, this is not a perfect solution but it does solve the issue
consider datum transformation when pasting features (fixes qgis#16846)
…_area

[processing] fix SAGA LTR Catchment Area tools
[processing] fix QGIS import in PostGIS optional parameter (fix qgis#16871)
Incorrect creation of geo transform was leading to invalid raster
outputs and many "creating warped vrt" log messages

(cherry-picked from 5241c74)
[processing] fix SAGA LTR mosaiking tool
@alexbruy
Copy link
Contributor

manuall merged

@alexbruy alexbruy closed this Jul 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet