format match_rule.py and chat.py#19
Merged
Merged
Conversation
The default GlobalStandard capacity of 50 (= 50K TPM) is too low for a
multi-step agent template. A single daily-briefing run pulls a chunk of
inbox, reasons over it, and renders -- easily 10-20K tokens. Two runs
within a minute trip the per-deployment rate limit and surface as:
HTTP 500: 'rate_limit_exceeded' ... 'Too Many Requests'
...which is a terrible first-deploy UX (people assume their config or
code is broken).
150 (= 150K TPM):
- Comfortably handles repeated single-user agent runs.
- Stays well within the default per-region/per-model GlobalStandard
quota for new subscriptions (typically 1000 units = 1M TPM/region
for the gpt-5 family).
- Leaves room in a sub to host several template deploys side-by-side
before quota-bumping is needed.
The param remains tweakable for users who are quota-constrained or who
need more headroom for multi-user workloads. To check your headroom:
az cognitiveservices usage list --location <region> \
--query "[?contains(name.value, 'gpt-5.4-mini')]" -o table
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot stopped work on behalf of
manvkaur due to an error
June 16, 2026 22:25
manvkaur
changed the title
manvkaur
changed the base branch from
paulyuk/model-capacity-default-150
to
main
manvkaur
changed the base branch from
main
to
paulyuk/model-capacity-default-150
manvkaur
changed the base branch from
paulyuk/model-capacity-default-150
to
main
manvkaur
approved these changes
Jun 16, 2026
paulyuk
approved these changes
Jun 16, 2026
paulyuk
left a comment
paulyuk
left a comment
Member
There was a problem hiding this comment.
LGTM. I assume these are formatting changes more than functional? it was hard to tell at first because description and comments werent there and this is a stacked PR versus inline to my PR.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Thanks for the feedback on #18. I've created this new PR, which merges into #18, to address your comment. I will work on the changes and keep this PR's description up to date as I make progress.
Original PR: #18
Triggering comment (#18 (comment)):