Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Inference Scripts in README to Match Project Structure #313

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

xisen-w
Copy link

@xisen-w xisen-w commented Apr 1, 2024

Summary:

This PR introduces updates to the inference scripts, ensuring they accurately reflect the current project structure. Previously, users encountered errors when attempting to run inference due to outdated script paths and configurations. This update aims to streamline the user experience, enabling straightforward execution of inference scripts without the need for troubleshooting or modifications.

Changes:

Updated the paths within the inference scripts to match the latest project directory structure.
Ensured that all referenced configurations in the scripts are accurate and up-to-date, reflecting the current setup of the project.
Modified script examples in the documentation to prevent confusion and guide users more effectively.
Testing:
The changes have been tested with the following commands, confirming that the scripts now run smoothly without errors:

python -m scripts.animate --config configs/prompts/1-ToonYou.yaml
python -m scripts.animate --config configs/prompts/3-RcnzCartoon.yaml
python -m scripts.animate --config configs/prompts/v1/v1-1-ToonYou.yaml
python -m scripts.animate --config configs/prompts/v3/v3-3-RcnzCartoon.yaml

Impact:

These updates significantly improve the usability of the inference scripts, reducing potential friction for new users and contributing to a more intuitive getting-started process. By aligning the scripts with the current project structure, we also minimize the risk of future discrepancies as the project evolves.

Request for Review:

I invite reviewers to test the updated scripts in their environments to verify the improvements and suggest any further refinements. Your feedback is invaluable to ensure that these changes meet our quality standards and user expectations.

Updated the script for inference command
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