Repository navigation
Catalogue the fal models reelee-web uses (#82) - #83
Merged
Merged
Conversation
…1.5, flux-pro (#82) Priced from fal's public model pages (read 2026-10-13); kling v1.5 standard has no published price and is flagged approximate. New entries carry quality_tier 'unranked' so no default pick changes. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
… test alias uniqueness (#82) Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #82.
Adds the reelee-web fal models to the catalogue, each with a
cost_estimateread from fal's public model pages on 2026-10-13 (the pricing API needs a key, which this session did not have).fal-ai/ltx-video,.../image-to-videofal-ai/mochi-v1fal-ai/kling-video/v1.5/pro/{image,text}-to-videofal-ai/kling-video/v1.5/standardfal-ai/flux-proFindings for reelee-web's table:
fal-ai/flux-pro/v1.1andfal-ai/flux-pro/v1.1-ultrawere already catalogued, so the issue's list was shorter than stated.fal-ai/kling-video/v1.5/proand.../standardare not real fal endpoints (fal's model list returns no metadata for them). The real ones are.../pro/image-to-videoand.../pro/text-to-video. The bare pro id is an alias of the image-to-video record, so reelee'sget_modelstill resolves it.Compatibility: records are appended with
quality_tier="unranked"(new tier value, documented onModelRecord), sopick_modeldefaults do not move (test added). Regeneratedschema/and the skill files, which are checked against the catalogue.Tests:
tests/test_reelee_web_catalogue.py(no network); full suite 676 passed.reelee needs: a falaw release, then
git pull/ upgrade falaw in the deployed env; no code change. Then reelee-web's table can be derived from falaw.Adversarial review (fresh Opus subagent): OK to merge after one fix. Prices re-checked against fal's pages; default picks unchanged; all 10 reelee-web ids resolve and price a 5 s clip. Confirmed and fixed here: the new
flux-proalias duplicated one onflux-pro/v1.1(unreachable; dropped, and an alias-uniqueness test added);single_character_recommendedon the new models had no evidence (removed). Noted, not blocking: per-megapixel models quote ~0.59 MP by default while fal rounds up to a whole megapixel (true of every per-megapixel record, not new);video_model_constraints()now lists the five added video models; the bare kling v1.5 ids are not fal endpoints, so the alias is a best guess, and nothing in reelee-web requests them.