Skip to content

Conversation

@PaulWessel
Copy link
Member

Description of proposed changes

So far, grdmix and three input grid insisted on the -Ccpt option but if the three grids are red, green, blue grids in 0-255 range there is no need to force -C. This PR allows -C to be optional if given components.

All tests are unaffected.

@PaulWessel PaulWessel added enhancement Improving an existing feature add-changelog Add PR to the changelog labels Jan 19, 2024
@PaulWessel PaulWessel added this to the 6.6.0 milestone Jan 19, 2024
@PaulWessel PaulWessel requested a review from a team January 19, 2024 11:58
@PaulWessel PaulWessel self-assigned this Jan 19, 2024
@PaulWessel PaulWessel merged commit 634e898 into master Jan 19, 2024
@PaulWessel PaulWessel deleted the grdmix-rgb branch January 19, 2024 12:37
remkos added a commit that referenced this pull request Jan 23, 2024
* 'master' of https://github.com/GenericMappingTools/gmt: (24 commits)
  Bump actions/cache from 3 to 4 (#8323)
  Add longoptions support for psbarb. (#8283)
  Add longoptions support for nearneighbor. (#8281)
  Add longoptions support for makecpt. (#8280)
  Add longoptions support for kml2gmt. (#8279)
  Add longoptions support for inset. (#8278)
  Add longoptions support for grdvolume. (#8276)
  Add longoptions support for grdview. (#8275)
  Batch #8 of longoptions translation tests. (#8272)
  Enable longoptions support for grdconvert. (#8197)
  Enable longoptions support for grd2xyz. (#8178)
  Update gmt_init.c
  Allow -A even if movie is not generated
  typos
  Update std-opts.rst
  Same case, clearer titles
  Dateline crossing dcw polygons and tiles (#8318)
  Missed a space before -T synopsis
  Improve windbarb docs, synopsis and usage for -Q and parsing of -Q (#8314)
  Let grdmix create RGB imagea for red, green, blue grids in 0-255 range (#8316)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

add-changelog Add PR to the changelog enhancement Improving an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants