Skip to content

Conversation

@darrylong
Copy link
Member

Description

This PR adds a section in the readme to showcase an AB-testing framework which uses the Cornac API serving extension.

Related Issues

Checklist:

  • I have added tests.
  • I have updated the documentation accordingly.
  • I have updated README.md (if you are adding a new model).
  • I have updated examples/README.md (if you are adding a new example).
  • I have updated datasets/README.md (if you are adding a new dataset).

@darrylong darrylong requested a review from qtuantruong July 18, 2024 04:57
@darrylong darrylong self-assigned this Jul 18, 2024
Copy link
Member

@qtuantruong qtuantruong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@darrylong darrylong added the docs Documentation (Readme, readthedocs) related label Jul 22, 2024
@qtuantruong
Copy link
Member

@darrylong any anticipated changes to this? If not, shall we merge this PR?

@darrylong
Copy link
Member Author

darrylong commented Sep 4, 2024

@darrylong any anticipated changes to this? If not, shall we merge this PR?

No anticipated changes for now. Let's merge this.

@darrylong darrylong merged commit 73123c9 into PreferredAI:master Sep 4, 2024
@darrylong darrylong deleted the update-readme-cornacab branch September 4, 2024 03:39
qtuantruong pushed a commit to qtuantruong/cornac that referenced this pull request Oct 25, 2025
* Add Cornac-AB section to Readme.md

* Update README.md

* Update assets directory and downsized images

---------

Co-authored-by: Tuan Truong <[email protected]>
qtuantruong pushed a commit to qtuantruong/cornac that referenced this pull request Oct 25, 2025
* Add Cornac-AB section to Readme.md

* Update README.md

* Update assets directory and downsized images

---------

Co-authored-by: Tuan Truong <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation (Readme, readthedocs) related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants