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

Add missing options in pipeline API #884

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

Conversation

gyagp
Copy link

@gyagp gyagp commented Aug 8, 2024

This is to add missing options in pipeline API to align with from_pretrained API.
Bug #883 can be fixed with this. That model has a separate data file, so use_external_data_format has to be supported and designated to true.

@gyagp gyagp changed the base branch from main to v3 August 8, 2024 08:53
src/pipelines.js Outdated
@@ -1,15 +1,15 @@
/**
* @file Pipelines provide a high-level, easy to use, API for running machine learning models.
*
*
Copy link
Author

Choose a reason for hiding this comment

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

My editor trims the tailing spaces. Please let me know if you don't like this change. For review, you may set the option to "Hide whitespace".

@gyagp
Copy link
Author

gyagp commented Aug 8, 2024

@xenova Please review this change, thanks!

@xenova
Copy link
Collaborator

xenova commented Aug 9, 2024

Thanks! To keep the diff small (and consistent across other files), can you add back the whitespace for those lines (I am adding prettier which should fix this later)?

@gyagp
Copy link
Author

gyagp commented Aug 9, 2024

Thanks! To keep the diff small (and consistent across other files), can you add back the whitespace for those lines (I am adding prettier which should fix this later)?

Updated.
Sorry that my local changes are a bit messy, so I had to do a force push. I think it's because I resolved some merge conflicts using GitHub directly.

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.

2 participants