-
Notifications
You must be signed in to change notification settings - Fork 4.5k
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 trust_remote_code Parameter and Set Default to False #5819
Conversation
Can we instead set trust_remote_code to True by default? As many more advanced models are getting their own vibes these days. So we have one less thing to worry about? lol. |
@hohoCode it is set to True, by default, but I am suggesting to remove the default behavior. |
could you rebase this pr? https://github.com/hiyouga/LLaMA-Factory/blob/main/.github/CONTRIBUTING.md |
done! |
- Introduced a new model parameter `trust_remote_code` - Set the default value of `trust_remote_code` to `False` to enhance security
ecdb2bb
to
36f14f9
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for your contribution
What does this PR do?
trust_remote_code
trust_remote_code
toFalse
to enhance securityBefore submitting