Skip to content

Restructure - #285

Merged
InftyAI-Agent merged 4 commits into
InftyAI:mainfrom
kerthcet:restructure
Oct 25, 2026
Merged

InftyAI-Agent merged 4 commits into
InftyAI:mainfrom
kerthcet:restructure

Conversation

@kerthcet

Copy link
Copy Markdown
Member

What this PR does / why we need it

Which issue(s) this PR fixes

Fixes #

Special notes for your reviewer

Does this PR introduce a user-facing change?


@InftyAI-Agent InftyAI-Agent added needs-triage Indicates an issue or PR lacks a label and requires one. needs-priority Indicates a PR lacks a label and requires one. do-not-merge/needs-kind Indicates a PR lacks a label and requires one. approved Indicates a PR has been approved by an approver from all required OWNERS files. labels Oct 25, 2026
Signed-off-by: kerthcet <kerthcet@gmail.com>
@kerthcet

Copy link
Copy Markdown
Member Author

/lgtm
/kind cleanup

@InftyAI-Agent InftyAI-Agent added lgtm Looks good to me, indicates that a PR is ready to be merged. cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. and removed do-not-merge/needs-kind Indicates a PR lacks a label and requires one. labels Oct 25, 2026
kerthcet and others added 2 commits October 25, 2026 20:31
gen_ai.usage.input_tokens already includes cached tokens for both the
OpenAI and Anthropic instrumentations, but token totals added the cache
read and creation tokens on top, and cost charged cached tokens at both
the regular input price and the cache price.

Compute totals as input + output, and only charge the regular input
price for uncached input tokens.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- Remove test_model.py (its only test was commented out), the unused
  GraphQL test helper, and the unused graphql_context and
  setup_test_team fixtures.
- Drop test_tracking.py's copies of the shared conftest fixtures.
- Drop tests fully covered by others: test_push_with_empty_folder and
  five delete tests in test_sql.py.
- Merge the experiment resume and status tests into two parametrized
  tests.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@InftyAI-Agent InftyAI-Agent removed the lgtm Looks good to me, indicates that a PR is ready to be merged. label Oct 25, 2026
Signed-off-by: kerthcet <kerthcet@gmail.com>
@kerthcet

Copy link
Copy Markdown
Member Author

/lgtm

@InftyAI-Agent InftyAI-Agent added the lgtm Looks good to me, indicates that a PR is ready to be merged. label Oct 25, 2026
@InftyAI-Agent
InftyAI-Agent merged commit aab3e48 into InftyAI:main Oct 25, 2026
11 checks passed
@kerthcet
kerthcet deleted the restructure branch October 26, 2026 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. cleanup Categorizes issue or PR as related to cleaning up code, process, or technical debt. lgtm Looks good to me, indicates that a PR is ready to be merged. needs-priority Indicates a PR lacks a label and requires one. needs-triage Indicates an issue or PR lacks a label and requires one.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants