-
Notifications
You must be signed in to change notification settings - Fork 161
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
CePO #152
CePO #152
Conversation
Hi, thanks for contributing your changes upstream. Can we organize the code in its own cepo folder under You can put a detailed README.md in that folder as well along with the configs/ We can keep the main results table under the https://github.com/codelion/optillm?tab=readme-ov-file#sota-results-on-benchmarks-with-optillm and move the rest of the details into its own README. |
This will allow users to do the following: 1. Cli where if they can pass in anything that's "cepo_<name-of-attribute>" 2. Yaml file where if they pass it in as "<name-of-attribute>" 3. If none of them have a specific attribute, we use the default setting 4. If both of them have the specific attribute, we error out
…ass for single source of truth
Added a license header
Added a link to view the file changes
Added a license header
Added a license header
984eeaa
to
888bfd1
Compare
Thank you, @codelion! Certainly! I moved the code, config, and the CePO part of README into optillm/cepo, I only left the results for CePO vs base models in the SOTA results section. |
Looks, good. I will merge the PR. Thank you for your contributions. I will also cut a new release of the PYPI package with these changes later today. |
Added CePO as an optional method: