- Notifications
You must be signed in to change notification settings - Fork770
Move topk file#5240
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Merged
Uh oh!
There was an error while loading.Please reload this page.
Merged
Move topk file#5240
Changes from1 commit
Commits
Show all changes
81 commits Select commitHold shift + click to select a range
49a294e Initial partial implementation of betting-based confidence sequences
amishler81d4ad6 Fix bets to use previous time step's variance estimate
amishlerace5568 Add updates to wealth processes
amishlerc4f4a56 Add last step of confidence sequence calculation from hedged wealth p…
amishlere350c5e Change product to log-sum-exp for numerical stability
amishler2290c1d Add computation of point estimator for the mean
amishlerba8f9fc Combine weighting and hedged wealth process computation
amishlerfbf8cd9 Move bet calculation to helper function, add bet truncation
amishler2e2e85d Tweak comment string for clarity
amishler103f835 Add module-level and type-level docstrings
amishler35465e8 Remove max vs combo hedging choice, explain use of max in docstring
amishler4400783 For computing mean estimate, restrict search to inside confidence int…
amishler61705c0 Add unit tests, fix search for interval endpoints
amishlerf262d08 Tweak test and test comments
amishler8d9f55b Fix regularized mean value, add regression tests with known values
amishlera5ffaa0 Move test, clarify comment strings in another test
amishler8e7561b Add test with known confidence sequence values
amishlerd7781d4 Change tests to use non-constant observations for variance fluctuations
amishler79763d3 Merge branch 'main' into alan/betting-confidence-sequences
amishlera8ba087 Rename module to contrast with asymptotic_confidence_sequences.rs
amishler1e00912 Add input validation and associated tests
amishler45b13ad Remove unnecessary iterator method for m-values
amishlerb4a1c37 Make return type a Result, use anyhow for errors
amishler523722c Create enums and structs
amishler1bf28c9 Add check_topk_stopping(), allow dead code as needed
amishler86e195e Add epsilon argument, changed from allow to expect dead_code
amishler6553b59 Add wip note to docstring
amishler6210fe3 Add tests using non-zero epsilon tolerance
amishler6865c10 Add more tests with k_min not equal to k_max
amishler893f731 Add docstrings to tests
amishlerd92a7ac Change return value when k_min > num_variants
amishler1d78a4b Tweak docstrings
amishlerbf848a1 Remove enum that will be included in a separate PR
amishlerc7860a6 Add VariantStatus enum
amishler8a2b205 Refactor evaluations to accept a vector of variants for batch processing
amishlere602ebe Remove changes from topk.rs for now
amishlerb313209 Use partition_point() instead of binary search for finding confidence…
amishler1ccb1ba Make enum for specifying grid of points where wealth processes are ca…
amishlerc62b891 Move argument validation for WealthProcessGridPoints to new construct…
amishler6868cd4 Fix bug where variant could count itself when checking how many varia…
amishler69faf6e Add variant_names option to CLI and python client
amishler54fc560 Merge branch 'alan/betting-confidence-sequences' into alan/topk-enums…
amishlere1aeddc Fix tests to use new WealthProcessGridPoints enum
amishlerf7187e0 Don't multiply num_datapoints by num_variants
amishler3790058 Merge branch 'alan/topk-enums-structs-stopping' into alan/evals-batch…
amishlerab3f5fd Update some tests to use variant_names instead of variant_name
amishler89ff201 Merge branch 'alan/evals-batch-variants' of github.com:tensorzero/ten…
amishler726346e Merge branch 'main' into alan/evals-batch-variants
amishler644cfa4 Add deprecation warning for variant_name argument
amishler822ac6e Add deprecation warning for variant_name arg to python client
amishler02bd19e Merge branch 'alan/evals-batch-variants' of github.com:tensorzero/ten…
amishler0c28a29 Factor out batch evals functionality into helper function
amishlerc0b0517 Revert change to allow passing multiple variants to run_evaluations()
amishlera5af109 Merge branch 'main' into alan/evals-batch-variants
amishlere5b3727 Add variant back into tracing span
amishler28691f9 Merge branch 'alan/evals-batch-variants' of github.com:tensorzero/ten…
amishler9252e61 Add initial top-k orchestrator types and compute_updates() placeholder
amishlerd9b165b Remove redundancy from top-k batch processing
amishler3245107 Remove some types that will be used in run_topk() for a future PR
amishlerb31c6d2 Replace fully qualified paths with imports
amishler9942a02 Clarify scoring function trait, change compute_updates() to use exter…
amishlera668928 Change return type of process_topk_batch() to minimize computation re…
amishler5ab72ce Change function signature for process_topk_batch in anticipation that…
amishlerffb9c1d Correct arg name in docstring
amishler67a0897 Add debug checks that scores are bounded in [0, 1]
amishlerc1b7cbb Tweak docstring
amishler539e26e Remove batch processing, organize types into sections
amishler051bacd Remove TopKContext and extra imports
amishlerb5e7085 Add remaining core types
amishlerb070651 Add unit tests for compute_updates with non-empty evaluation results
amishler6f4a913 Merge branch 'main' into alan/evals-topk-batch-processing
amishlercb73452 Swap order of enum and struct
amishler701b2f9 Rename AdaptiveEvalStoppingResults to TopKTaskOutput
amishlerc8b0cba Expand unit tests for compute_updates(), move to top of tests mod
amishler249f6e6 Throw error if a task gets cancelled
amishler5c92bce Add top-k tie-breaking logic and related unit tests
amishlerb73b4e0 Create struct to replace long tuple
amishler93fa478 Move topk file
amishler792c075 Rename topk file
amishlerfe1579e Add module back to lib.rs
amishlerec5ea7d Merge branch 'main' into alan/evals-split-file
amishlerFile filter
Filter by extension
Conversations
Failed to load comments.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Jump to
Jump to file
Failed to load files.
Loading
Uh oh!
There was an error while loading.Please reload this page.
Diff view
Diff view
Add more tests with k_min not equal to k_max
- Loading branch information
Uh oh!
There was an error while loading.Please reload this page.
commit6865c10190b62ea4fad5cc4fbea84157ddf9d8d4
There are no files selected for viewing
171 changes: 128 additions & 43 deletionsevaluations/src/topk.rs
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
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.