docs: clarify video material sources - #1190
Conversation
|
Thank you for the clear explanation of how stock, local, and AI-generated materials flow through the shared production pipeline. The documentation direction is useful, and I verified that the proposed diff does not alter any sponsor content. The branch is now out of date and the patch no longer applies cleanly to the current README files. Since this PR was opened, main has added WaveSpeed AI as another text-to-video material source and continues to maintain a Japanese README, so merging the current text would make the new overview incomplete across both supported sources and languages. Could you please rebase onto the latest main, cover both WaveSpeed AI and Shengsuan Cloud / LoomLoom in the overview, preserve the current feature-list wording and sponsor sections, and add the equivalent explanation to README-ja.md? Please keep the change documentation-only and focused. Thank you again; we will be happy to review the updated version. |
Why: - Distinguish stock retrieval and local editing from paid AI video generation. - Keep the overview aligned with the current WaveSpeed and Shengsuan Cloud integrations. What: - Explain all video material sources in the Chinese, English, and Japanese READMEs. - Document charge confirmation for WaveSpeed and LoomLoom while preserving the existing feature and sponsor sections. Risk: - Documentation-only change; runtime behavior is unchanged. Tests: - git diff --check upstream/main - uv run --no-sync python -m compileall -q app cli.py main.py webui test - uv run --no-sync ruff check app cli.py main.py webui test - uv run --no-sync python -X utf8 -m coverage run -m pytest -q test (712 passed, 11 skipped, 6084 subtests passed) Live Docs: - Not applicable.
1851d48 to
751c6c7
Compare
|
Updated as requested—thank you for the detailed review.
The PR is now mergeable, and the refreshed CI jobs are in progress. |
Summary
Why
The introduction can lead readers to expect that every workflow synthesizes new video frames. The new overview makes the selected video source and shared post-production pipeline explicit, sets license expectations for stock and local material, and covers both AI video integrations currently available on
main. It also clarifies the source question raised in #1181 without claiming support for arbitrary creator or channel URLs.Testing
upstream/main(05c0baf)README.md,README-en.md, andREADME-ja.mdgit diff --check upstream/main...HEADuv run --no-sync python -m compileall -q app cli.py main.py webui testuv run --no-sync ruff check app cli.py main.py webui testuv run --no-sync python -X utf8 -m coverage run -m pytest -q test: 712 passed, 11 skipped, 6084 subtests passedCloses #1183