Skip to content

Conversation

@hrodmn
Copy link
Contributor

@hrodmn hrodmn commented Sep 20, 2024

If someone is using the /cog/viewer endpoint to figure out which render parameters make sense for their dataset it would be nice to add a Show Render Parameters button. It would also be nice to be able to share a link to a map with those parameters applied so the user could send someone a link to the exact image that they decided on. It might be better to link to /cog/viewer since /cog/map is deprecated, but right now you can't set render parameters in that endpoint (just url).

image

This idea came from Team 4 in the IMPACT X-athon with @anayeaye @smk0033 @unhipcat

@vincentsarago
Copy link
Member

🥳
thanks for starting this @hrodmn

Would you be able to change few things, mostly making list parameters as an array [] in the render object:

  • "rescale": "0,1000" -> "rescale": [[0,1000]]
  • "bidx": 1 -> "bidx": [1]

@vincentsarago
Copy link
Member

vincentsarago commented Sep 20, 2024

also /cog/map is deprecated but not /cog/{tileMatrixSetId}/map ;-)

but right now you can't set render parameters in that endpoint (just url).

We could update the code to accept more parameters ... but to be honest I would prefer if we re-write the whole application 😅

@hrodmn
Copy link
Contributor Author

hrodmn commented Sep 20, 2024

We could update the code to accept more parameters ... but to be honest I would prefer if we re-write the whole application 😅

It would be great to move away from the giant html template file someday, but it works!

@vincentsarago vincentsarago changed the base branch from main to maint-0.18.x September 23, 2024 13:07
@vincentsarago vincentsarago changed the base branch from maint-0.18.x to main September 23, 2024 13:10
@vincentsarago vincentsarago merged commit 6e43eec into developmentseed:main Sep 23, 2024
10 checks passed
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.

2 participants