Skip to content

Label detailed usage costs as estimated - #389

Merged
andy-xu-db merged 1 commit into
usage-warehouse-pickerfrom
estimated-usage-cost-labels
Aug 31, 2026
Merged

Label detailed usage costs as estimated#389
andy-xu-db merged 1 commit into
usage-warehouse-pickerfrom
estimated-usage-cost-labels

Conversation

@andy-xu-db

@andy-xu-db andy-xu-db commented Aug 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • label the per-model cost column as Est. Cost (USD)
  • label tool-level cost totals as Est. Cost (USD)
  • leave cost calculations unchanged

Testing

  • uv run pytest tests/test_usage.py (87 passed)
  • uv run ruff check src/ucode/usage.py tests/test_usage.py

🥞 Stacked PR

Use this link to review incremental changes.


@andy-xu-db
andy-xu-db marked this pull request as ready for review August 25, 2026 15:13
@andy-xu-db
andy-xu-db force-pushed the estimated-usage-cost-labels branch from 803bd76 to 5525ee7 Compare August 25, 2026 20:25
@andy-xu-db
andy-xu-db force-pushed the usage-warehouse-picker branch 2 times, most recently from f952464 to 9ef700d Compare August 25, 2026 20:57
@andy-xu-db
andy-xu-db force-pushed the estimated-usage-cost-labels branch from 5525ee7 to d5c24eb Compare August 25, 2026 20:57
@andy-xu-db
andy-xu-db force-pushed the usage-warehouse-picker branch from 9ef700d to 3663d3d Compare August 25, 2026 20:57
@andy-xu-db
andy-xu-db force-pushed the estimated-usage-cost-labels branch from d5c24eb to 340226f Compare August 25, 2026 20:57
@andy-xu-db
andy-xu-db merged commit 1ac19fb into usage-warehouse-picker Aug 31, 2026
2 checks passed
@andy-xu-db
andy-xu-db deleted the estimated-usage-cost-labels branch August 31, 2026 21:53
andy-xu-db added a commit that referenced this pull request Aug 31, 2026
* Let users choose a usage SQL warehouse

* Label detailed usage costs as estimated (#389)

Co-authored-by: andy-xu-db <310751426+andy-xu-db@users.noreply.github.com>

* Reprompt after usage warehouse failure

---------

Co-authored-by: andy-xu-db <310751426+andy-xu-db@users.noreply.github.com>
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