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

refactor: show progress while encoding #586

Merged
merged 3 commits into from
Oct 24, 2022
Merged

refactor: show progress while encoding #586

merged 3 commits into from
Oct 24, 2022

Conversation

bwanglzu
Copy link
Member

@bwanglzu bwanglzu commented Oct 24, 2022

Usability improvements encode:

  1. interpret device type if device is not provided.
  2. show_progress while batching DocumentArray when calling encode.

  • This PR references an open issue
  • I have added a line about this change to CHANGELOG

@bwanglzu bwanglzu marked this pull request as ready for review October 24, 2022 10:15
@bwanglzu bwanglzu self-assigned this Oct 24, 2022
@@ -22,6 +22,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

- Update the open clip model names in the table of the backbones. ([#580](https://github.com/jina-ai/finetuner/pull/580))

- Show progress while encode batches of documents. ([#586](https://github.com/jina-ai/finetuner/pull/586))
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

while encoding

Comment on lines +352 to +354
if not device:
device = 'cuda' if torch.cuda.is_available() else 'cpu'

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should I add this change to the build model function in pr #584?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes please

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants