Skip to content

feat: interpolate {datetime} in if sel includes {dim}={datetime}#78

Merged
hrodmn merged 4 commits intodevelopfrom
fix/interpolate-sel-datetime
Oct 3, 2025
Merged

feat: interpolate {datetime} in if sel includes {dim}={datetime}#78
hrodmn merged 4 commits intodevelopfrom
fix/interpolate-sel-datetime

Conversation

@hrodmn
Copy link
Contributor

@hrodmn hrodmn commented Sep 23, 2025

Before this change, it was impossible to use the time series API for datasets that needed sel: time={timestamp}. I originally tried adding a time series API query parameter to explicitly pass the datetime parameter onto the array of sel values (#76), but it was clunky and I thought a string interpolation approach would be more flexible and easier to specify in downstream applications.

Now if a user provides time={datetime} in the sel array, titiler-cmr will automatically pluck the datetime query parameter value and interpolate it into the time= string. I put some examples in the notebooks.

I think this is a good idea for titiler-cmr because temporal queries are a critical component of the whole application and extending the datetime parameter to be useful within the sel context will make it more useful for at least two datasets 😅 .

@github-actions
Copy link

github-actions bot commented Sep 23, 2025

📚 Documentation preview will be available at: https://developmentseed.github.io/titiler-cmr/pr-previews/pr-78/

Status: ✅ Preview is ready!

github-actions bot pushed a commit that referenced this pull request Sep 23, 2025
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

github-actions bot pushed a commit that referenced this pull request Sep 23, 2025
@hrodmn hrodmn marked this pull request as ready for review September 23, 2025 15:04
github-actions bot pushed a commit that referenced this pull request Sep 23, 2025
@hrodmn hrodmn merged commit ef8c494 into develop Oct 3, 2025
7 checks passed
@hrodmn hrodmn deleted the fix/interpolate-sel-datetime branch October 3, 2025 11:25
github-actions bot pushed a commit that referenced this pull request Oct 3, 2025
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.

3 participants