Skip to content

fix: rilltime syntax in comparison time range in citation urls#9478

Draft
AdityaHegde wants to merge 7 commits into
mainfrom
fix/rill-time-comparison-time-in-citation-urls
Draft

fix: rilltime syntax in comparison time range in citation urls#9478
AdityaHegde wants to merge 7 commits into
mainfrom
fix/rill-time-comparison-time-in-citation-urls

Conversation

@AdityaHegde
Copy link
Copy Markdown
Collaborator

@AdityaHegde AdityaHegde commented May 21, 2026

LLM can use rill time for relative time ranges. Since backend supports it in comparison time range as well, it can use it in comparison as well. But UI doesnt support rill time, leading to removal of comparison all together.

Since watermark history at the time of query is not available, we are opting to resolve the time ranges and store it in the tool result.

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

@AdityaHegde AdityaHegde force-pushed the fix/rill-time-comparison-time-in-citation-urls branch 3 times, most recently from d8044db to 4124b7c Compare May 21, 2026 11:27
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.

1 participant