Skip to content

Relax fireworks-ai dependency pin#432

Merged
benjibc merged 2 commits intomainfrom
benjibc/relax-fireworks-ai-pin
Mar 7, 2026
Merged

Relax fireworks-ai dependency pin#432
benjibc merged 2 commits intomainfrom
benjibc/relax-fireworks-ai-pin

Conversation

@benjibc
Copy link
Contributor

@benjibc benjibc commented Mar 7, 2026

Summary

  • relax the fireworks-ai dependency from an exact pin to >=1.0.0a20,<2

Why

  • eval-protocol==0.3.22 currently pins fireworks-ai==1.0.0a20
  • fireworks-training-cookbook now requires fireworks-ai>=1.0.0a36,<2
  • that makes cookbook CI fail at install time before any tests run

Validation

  • ./.venv/bin/python -m build
  • confirmed wheel metadata contains Requires-Dist: fireworks-ai<2,>=1.0.0a20
  • ./.venv/bin/pip install --dry-run dist/*.whl /Users/bennychen/Documents/cookbook/training resolves successfully and selects fireworks-ai-1.0.0a37

Note

Low Risk
Dependency constraint-only change with no runtime logic modifications; main risk is potential behavior changes when newer fireworks-ai versions are selected.

Overview
Relaxes the fireworks-ai dependency from an exact pin (==1.0.0a20) to a compatible range (>=1.0.0a20,<2) in pyproject.toml.

Updates uv.lock to reflect the new fireworks-ai version constraint in the resolved metadata.

Written by Cursor Bugbot for commit e95a396. This will update automatically on new commits. Configure here.

@benjibc benjibc merged commit 5ac0bb4 into main Mar 7, 2026
15 of 17 checks passed
@benjibc benjibc deleted the benjibc/relax-fireworks-ai-pin branch March 7, 2026 01:59
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.

1 participant