Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jun 3, 2025. It is now read-only.

[Transformers] Add an argument trust_remote_code on transformers pipeline initialization.#1173

Merged
dbogunowicz merged 1 commit into
mainfrom
feature/damian/trust_remote_code
Aug 8, 2023
Merged

[Transformers] Add an argument trust_remote_code on transformers pipeline initialization.#1173
dbogunowicz merged 1 commit into
mainfrom
feature/damian/trust_remote_code

Conversation

@dbogunowicz

Copy link
Copy Markdown
Contributor

Give a user an option to enable running remote code through pipeline argument e.g.

Pipeline.create(task="...", trust_remote_code = True)

By default trust_remote_code=False.

@dbogunowicz dbogunowicz requested review from bfineran and mgoin August 8, 2023 08:14

@mgoin mgoin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you!
Side note: I think there is a problem with discoverability for arguments that are inherited by pipeline. Maybe this would be "solved" by autogenerated docs but I'm not sure how to make it more visible just looking at the constructors

@rahul-tuli rahul-tuli left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! could you update the PR description with an example of what does this option allow a user to do that wasn't possible before?

@Satrat Satrat self-requested a review August 8, 2023 13:26
@dbogunowicz dbogunowicz merged commit c8f6b0e into main Aug 8, 2023
@dbogunowicz dbogunowicz deleted the feature/damian/trust_remote_code branch August 8, 2023 13:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants