Skip to content

[Promote] From Fireworks staging to cookbook - #699

Open
bot-fireworks-ai wants to merge 4 commits into
mainfrom
autogen/promote-cookbook-from-fireworks-main
Open

bot-fireworks-ai wants to merge 4 commits into
mainfrom
autogen/promote-cookbook-from-fireworks-main

Conversation

@bot-fireworks-ai

@bot-fireworks-ai bot-fireworks-ai commented Sep 23, 2026 •

Copy link
Copy Markdown
Contributor

Automated public repo promotion from fw-ai/fireworks.

Source staging path: public-repos/cookbook
Source staging SHA: 4fd6ded0fcc0ce71662429c3acf4a8b4cf149c3a

@Hecate0821 Hecate0821 added the autorelease: pending Autorelease promotion PR pending merge/tag finalization label Sep 23, 2026

@cursor cursor 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.

Stale comment

Risk: medium. Left a non-blocking comment and did not approve; human review is needed. The fireworks-ai training floor bump (1.2.14→1.2.15) also pulls pyqwest 0.6.2→0.10.0, so a human should confirm that transitive jump is intended and recipe-safe.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Approver

@cursor cursor 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.

Stale comment

Risk: medium. Left a non-blocking comment and did not approve; human review is needed. DPPO now changes divergence-specific default thresholds and drops TIS from the loss, so a human should confirm those training-objective defaults plus the fireworks-ai 1.2.15 / pyqwest 0.10.0 floor.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Approver

@cursor cursor 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.

Stale comment

Risk: medium. Left a non-blocking comment and did not approve; human review is needed. This head still changes DPPO defaults (per-divergence thresholds, TIS dropped from the loss), score-centering top-k limits, and an advantage_fn hook, so a human should confirm those training-objective defaults plus the fireworks-ai 1.2.15 floor.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Approver

@cursor cursor 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.

Risk: medium. Left a non-blocking comment and did not approve; human review is needed. This head still changes DPPO defaults (per-divergence thresholds, TIS dropped from the loss), score-centering top-k limits, and an advantage_fn hook, so a human should confirm those training-objective defaults plus the fireworks-ai 1.2.15 floor.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Approver

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autorelease: pending Autorelease promotion PR pending merge/tag finalization

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants