Skip to content

feat: add step3p5 for cce#3384

Merged
NanoCode012 merged 2 commits into
mainfrom
feat/step3p5
Feb 10, 2026
Merged

feat: add step3p5 for cce#3384
NanoCode012 merged 2 commits into
mainfrom
feat/step3p5

Conversation

@NanoCode012

@NanoCode012 NanoCode012 commented Feb 3, 2026

Copy link
Copy Markdown
Collaborator

Description

Add support in CCE for https://huggingface.co/stepfun-ai/Step-3.5-Flash

Motivation and Context

How has this been tested?

AI Usage Disclaimer

Screenshots (if appropriate)

Types of changes

Social Handles (Optional)

Summary by CodeRabbit

Release Notes

  • Chores

    • Updated cut-cross-entropy dependency reference.
  • Documentation

    • Updated installation commands across notebooks, scripts, and documentation.
    • Added new plugin entry to usage examples.

@coderabbitai

coderabbitai Bot commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto incremental reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

  • 🔍 Trigger a full review
📝 Walkthrough

Walkthrough

This PR updates the pinned git commit hash for the cut-cross-entropy dependency from f4b5712 to 0d4ce4b across installation instructions, scripts, and integration files. Additionally, a new plugin entry is added to the README usage documentation.

Changes

Cohort / File(s) Summary
Cut-cross-entropy commit hash update
examples/colab-notebooks/colab-axolotl-example.ipynb, scripts/cutcrossentropy_install.py, src/axolotl/integrations/cut_cross_entropy/__init__.py
Updated git commit reference from f4b5712 to 0d4ce4b for cut-cross-entropy dependency installation.
Cut-cross-entropy documentation update
src/axolotl/integrations/cut_cross_entropy/README.md
Updated installation command commit hash from f4b5712 to 0d4ce4b and added new plugin entry step3p5 to usage examples.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested labels

ready to merge

Suggested reviewers

  • winglian
  • SalmanMohammadi
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat: add step3p5 for cce' directly reflects the main change: adding support for the Step-3.5-Flash model in the cut-cross-entropy (CCE) integration, which is evident in the README update mentioning step3p5 plugin entry.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat/step3p5

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Failure to add the new IP will result in interrupted reviews.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@src/axolotl/integrations/cut_cross_entropy/README.md`:
- Line 93: The supported models list in README.md is out of alphabetical order:
move the `step3p5` entry so the sequence around `seed_oss`, `smollm3`,
`step3p5`, `voxtral` is alphabetically correct; specifically ensure `seed_oss`
is followed by `smollm3` and then `step3p5` (adjust the list entries near the
current `seed_oss`/`smollm3`/`voxtral` block to restore proper alphabetical
ordering).

- qwen3_next
- smollm3
- seed_oss
- step3p5

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟡 Minor

Fix alphabetical ordering in supported models list.

The model step3p5 is inserted between seed_oss and voxtral, but the list appears to be alphabetically sorted. The correct alphabetical order should be: seed_oss, smollm3, step3p5. Currently, smollm3 appears before seed_oss at line 91-92, which breaks alphabetical sorting.

📝 Proposed fix to restore alphabetical order
 - qwen3_next
+- seed_oss
 - smollm3
-- seed_oss
 - step3p5
 - voxtral
🤖 Prompt for AI Agents
In `@src/axolotl/integrations/cut_cross_entropy/README.md` at line 93, The
supported models list in README.md is out of alphabetical order: move the
`step3p5` entry so the sequence around `seed_oss`, `smollm3`, `step3p5`,
`voxtral` is alphabetically correct; specifically ensure `seed_oss` is followed
by `smollm3` and then `step3p5` (adjust the list entries near the current
`seed_oss`/`smollm3`/`voxtral` block to restore proper alphabetical ordering).

@github-actions

github-actions Bot commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

📖 Documentation Preview: https://69821718bc306719dc4ef2a2--resonant-treacle-0fd729.netlify.app

Deployed on Netlify from commit 9435a43

@codecov

codecov Bot commented Feb 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@NanoCode012 NanoCode012 merged commit b6d3653 into main Feb 10, 2026
20 checks passed
@NanoCode012 NanoCode012 deleted the feat/step3p5 branch February 10, 2026 10:51
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.

2 participants