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

Added progress meter to create method. #58

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Shy
Copy link

@Shy Shy commented Jan 20, 2025

Checklist

  • Discussed idea or confident it's critical (See Contribute.md).
  • One feature/bug per PR (unless minor/related).
  • PR is from a feature branch, not main.
  • I ran at least one example to ensure the code works.
  • Checked linting/formatting (uv run ruff format && uv run ruff check).

Description

Hey folks. I wanted to be able to keep an eye on how long generation was taking so I added tqdm to the create method. Happy to swap to an alternative progress bar or do this differently. I just made this change for myself locally and figured it might be useful to y'all?

This is a 1 line change, with 1 new dependency.

image

Shy added 2 commits January 20, 2025 12:51
I think this is the right way to do this? I'm usually using pyenv with pip so no idea.
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