-
Notifications
You must be signed in to change notification settings - Fork 19
[FSSDK-11169] Implement Decision Service methods to handle CMAB #403
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
Merged
Mat001
merged 19 commits into
master
from
mpirnovar-cmab-decisionservice-methods-fssdk-11169
Jun 6, 2025
Merged
Changes from all commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
b8f9e9c
add decision service methods to support cmab
Mat001 dd9ff73
refactor composite exper service for cmab and add tests
Mat001 4a81f4a
update failing tests
Mat001 9e9b82f
update experiment cmab service
Mat001 b4f1751
fix lint errors
Mat001 607794a
more lint fixes
Mat001 7c59df3
traffic allocation in cmab
Mat001 73eb9d2
remove duplicate line
Mat001 25e5eec
refactor to support traffic allocation
Mat001 6afa8db
fixed cmab/service.go
Mat001 7d76a70
add unit tests
Mat001 a54cfb0
fix lint errors and failed test
Mat001 646428c
add package name
Mat001 b195285
remove adapter, make client handle cmab error, fix EntityID string
Mat001 cbf50ee
revert cmab service to cleaner version from previous pr
Mat001 07a835e
fix remianing comments
Mat001 daa7aab
fix linting
Mat001 6dec6d2
Merge branch 'master' into mpirnovar-cmab-decisionservice-methods-fss…
Mat001 1cf4a97
fix coveralls
Mat001 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.