Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(ds): Implement recalibration for sampling mode PROJECT #80241

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

jan-auer
Copy link
Member

@jan-auer jan-auer commented Nov 5, 2024

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 5, 2024
Copy link

codecov bot commented Nov 5, 2024

Codecov Report

Attention: Patch coverage is 41.25000% with 47 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
.../sentry/dynamic_sampling/tasks/recalibrate_orgs.py 40.90% 22 Missing and 4 partials ⚠️
...dynamic_sampling/tasks/helpers/recalibrate_orgs.py 17.39% 19 Missing ⚠️
src/sentry/models/options/organization_option.py 71.42% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #80241      +/-   ##
==========================================
+ Coverage   78.07%   78.10%   +0.02%     
==========================================
  Files        7188     7184       -4     
  Lines      317611   317324     -287     
  Branches    43759    43729      -30     
==========================================
- Hits       247972   247836     -136     
+ Misses      63299    63138     -161     
- Partials     6340     6350      +10     

* master: (67 commits)
  feat(dynamic-sampling): Sampling breakdown (#80304)
  feat(profiling): add organizations:continuous-profiling to the list of exposable features (#80236)
  chore(broadcasts): remove cta column from broadcast model (#80201)
  feat(dynamic-sampling): Use sample rates endpoint (#80235)
  feat(issues): Rearrange all events columns, sizes (#80296)
  fix(issues): All event table pagination counts (#80297)
  fix(issues): Preserve query parameters on all events close (#80295)
  feat(issues): Hide "comment" button until focused (#80283)
  fix(sentry-app): Adds better validation for invalid token request bodies (#80289)
  feat(workflow_engine): Add in hook for producing occurrences from the stateful detector (#80168)
  feat(issue summary) New structured issue summary design (#80273)
  feat(workflow_engine): Return status change messages when a stateful detector resolves (#80122)
  feat(insights): Add insights query date range footer hook (#80276)
  ref(crons): Switch to cronsim in sample data generator (#80278)
  feat(issue-details): Hide merged/similar issues for non-error issues (#80284)
  feat(issue summary) Update issue summary model (#80270)
  feat(crons): Add cronsim behind an option (#80271)
  fix(anomaly detection): add alerts analytics reqs to utils/analytics.tsx (#80281)
  feat(trace-explorer): Sort traces by timestamp in EAP (#80274)
  feat(workflow_engine): Implement basic evaluation in `DataCondition` (#80118)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: Backend Automatically applied to PRs that change backend components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant