Skip to content

feat: add xarray dataset coordinate endpoint - #1504

Merged
vincentsarago merged 2 commits into
mainfrom
feat/xarray-coordinate-values
Sep 18, 2026
Merged

vincentsarago merged 2 commits into
mainfrom
feat/xarray-coordinate-values

Conversation

@hrodmn

@hrodmn hrodmn commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Technical Context

Description

Adds /dataset/coordinates/{name} endpoint to retrieve coordinate values for an xarray dataset dimension


Checklist

  • Linting: Code is formatted and linted (uv run pre-commit run --all-files)
  • Tests: Tests pass. I have included new tests for these changes where applicable.
  • Edge Cases: I have manually verified "unhappy paths" and edge cases beyond the basic success criteria (e.g., database connection timeouts, malformed input, strict mapping rejections).
  • Documentation: I have updated README.md to reflect any new environment variables, configuration changes, or breaking updates.
  • Accountability: I can explain the implementation logic for every line of code submitted.

AI tool usage

  • AI (Copilot or something similar) supported my development of this PR. Use of AI tools HAVE TO be indicated.

@github-actions github-actions Bot 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.

⚠️ Performance Alert ⚠️

Possible performance regression was detected for benchmark 'TiTiler performance Benchmarks'.
Benchmark result of this commit is worse than the previous benchmark result exceeding threshold 1.30.

Benchmark suite Current: f8f0244 Previous: 9669764 Ratio
WebMercator longest_transaction 0.2 s 0.14 s 1.43

This comment was automatically generated by workflow using github-action-benchmark.

@hrodmn hrodmn self-assigned this Sep 17, 2026
@vincentsarago

Copy link
Copy Markdown
Member

thanks @hrodmn

might makes sense to also add /coordinates to return the list of available coordinates 🤷

@hrodmn

hrodmn commented Sep 18, 2026

Copy link
Copy Markdown
Contributor Author

might makes sense to also add /coordinates to return the list of available coordinates 🤷

I added /dataset/dimensions that returns {"dimension_name": dimension_size", ...}

@vincentsarago vincentsarago left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

🚀

@vincentsarago
vincentsarago merged commit bca60b5 into main Sep 18, 2026
11 checks passed
@vincentsarago
vincentsarago deleted the feat/xarray-coordinate-values branch September 18, 2026 14:46
@ds-release-bot ds-release-bot Bot mentioned this pull request Sep 18, 2026

This branch was successfully deployed

1 active deployment
ci — b1303e1f Deployed Sep 18, 2026 by hrodmn via tests (3.13) #69
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.

Expose xarray coordinate values

2 participants