Skip to content

Rast_log_colors: Another attempt to address r.colors -g/-a (#1480) - #1937

Merged
HuidaeCho merged 3 commits into
OSGeo:mainfrom
HuidaeCho:raster_log_colors_2
Oct 16, 2021
Merged

HuidaeCho merged 3 commits into
OSGeo:mainfrom
HuidaeCho:raster_log_colors_2

Conversation

@HuidaeCho

@HuidaeCho HuidaeCho commented Oct 11, 2021 •

Copy link
Copy Markdown
Member

Unlike #1481 or #1923, two other PRs trying to address #1480, this PR shifts cell values by a delta if the min is less than or equal to 0 such that the logs of shifted min and max can be calculated. Then, just before writing color rules, unshifted cell values are approximated by subtracting the delta.

@HuidaeCho HuidaeCho changed the title color_xform: Another attempt to address r.colors -g/-a (#1480) Rast_log_colors: Another attempt to address r.colors -g/-a (#1480) Oct 11, 2021
@HuidaeCho
HuidaeCho marked this pull request as draft October 11, 2021 05:03
@HuidaeCho
HuidaeCho marked this pull request as ready for review October 11, 2021 05:53

@susi susi left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah now this works! 🎉

@HuidaeCho
HuidaeCho merged commit 6376395 into OSGeo:main Oct 16, 2021
@HuidaeCho
HuidaeCho deleted the raster_log_colors_2 branch October 16, 2021 05:58
@neteler neteler added this to the 8.0.0 milestone Dec 9, 2021
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Oct 26, 2022
… (OSGeo#1937)

* color_xform: Another attempt to address r.colors -g/-a (OSGeo#1480)

* Fix absolute log

* Add a comment
ninsbl pushed a commit to ninsbl/grass that referenced this pull request Feb 17, 2023
… (OSGeo#1937)

* color_xform: Another attempt to address r.colors -g/-a (OSGeo#1480)

* Fix absolute log

* Add a comment
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.

[Bug] r.colors: -g (log) and -a (absolute log) don't work as expected

3 participants