Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

hassiebp
Copy link
Contributor

@hassiebp hassiebp commented Sep 15, 2025

Important

Skip tokenization and cost calculation for ERROR level observations in IngestionService.

  • Behavior:
    • In index.ts, skip tokenization and cost calculation in getUsageUnits() if observationRecord.level is ERROR.
  • Tests:
    • Add test case in calculateTokenCost.unit.test.ts to verify skipping of tokenization and cost calculation when level is ERROR.

This description was created by Ellipsis for f6494e6. You can customize this summary. It will automatically update as commits are pushed.

@dosubot dosubot bot added size:M This PR changes 30-99 lines, ignoring generated files. bug Something isn't working labels Sep 15, 2025
@hassiebp hassiebp requested a review from Steffen911 September 15, 2025 08:57
@hassiebp hassiebp linked an issue Sep 15, 2025 that may be closed by this pull request
@hassiebp hassiebp added this pull request to the merge queue Sep 15, 2025
@dosubot dosubot bot added the auto-merge This PR is set to be merged label Sep 15, 2025
Merged via the queue into main with commit 9a09fa2 Sep 15, 2025
29 checks passed
@hassiebp hassiebp deleted the hassieb/lfe-6639-skip-tokenization-ingestion-level-error branch September 15, 2025 13:25
@dosubot dosubot bot removed the auto-merge This PR is set to be merged label Sep 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: failed LLM requests should not count for usage?
1 participant