Skip to content

Conversation

@vincentsarago
Copy link
Member

match earth-mover/xpublish-tiles#164 (🙏 @dcherian)

Before this PR we used sel-method=nearest parameter but this limited the possibility to use different method for different dimension.

With this PR we introduce a new string format for selectors: sel={dimension}=({method}::)?{value}

@dcherian
Copy link

Haha I love that we're stealing ideas from each other. Hurray for open source!

@vincentsarago
Copy link
Member Author

I'm still a bit hesitant about the :: notation.

In other titiler dependencies we use pipe | to handle complexe query parameters (e.g asset_bidx={asset}|1,2,3) so maybe we should also use | here: sel={dimension}={value}(|{method})?

@abarciauskas-bgse
Copy link
Contributor

Do we have a sense of pros and cons for each syntax option (i.e. |, ::, others?) for selector parameters?

@vincentsarago
Copy link
Member Author

Do we have a sense of pros and cons for each syntax option (i.e. |, ::, others?) for selector parameters?

both are uncommon delimiters and would need to be URI encoded

@vincentsarago vincentsarago changed the title allow per selector method switch to sel={dim}={method}::{value} notation to specify selector method Nov 18, 2025
@vincentsarago
Copy link
Member Author

@dcherian can you share why you went with :: as separator?

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.

4 participants