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

Implement fixed and percentage sampling #109

Merged
merged 1 commit into from
Mar 16, 2025
Merged

Conversation

tginsberg
Copy link
Owner

No description provided.

+ Adding commons-statistics-inference to test scope to validate results are statistically accurate
Copy link

Overall Project 98.39% -0.13% 🍏
Files changed 97.11% 🍏

File Coverage
Gatherers4j.java 100% 🍏
SampleFixedSizeGatherer.java 98.99% -1.01% 🍏
GathererUtils.java 95.89% 🍏
SamplePercentageGatherers.java 91.11% -8.89% 🍏

@tginsberg tginsberg merged commit 9148179 into release/0.10.0 Mar 16, 2025
2 checks passed
@tginsberg tginsberg deleted the feature/sampling branch March 16, 2025 22:34
tginsberg added a commit that referenced this pull request Mar 19, 2025
+ Implement mapIndexed() (#102)
+ Use merge instead of compute in FrequencyGatherer
+ Documentation fix in changelog for mapIndexed()
+ Gatherers4j methods should return Gatherer interface when the implementation has no user-callable methods
+ Consistency fix: Rename IndexedValue to WithIndex (#105)
+ Reorganize packages (#106)
  - Reorg: Move utils to subpackage
  - Reorg: Move dtos to subpackage
  - Reorg: Move test utils
+ Implement peekIndexed() (#108)
+ Implement fixed and percentage sampling (#109)
+ Adding commons-statistics-inference to test scope to validate results are statistically accurate
+ Ensure bulk pushes to downstream listen for short-circuit signals (#110)
+ Upgrade to Java 24
  - This uses Corretto JDK for 24 in GitHub Actions until Temurin is out
  - Remove mentions of Java 23
  - Use JaCoCo Snapshot until full release
  - Upgrade Gradle to 8.13
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.

1 participant