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 leash mode #68

Merged
merged 6 commits into from
Feb 16, 2024
Merged

Add leash mode #68

merged 6 commits into from
Feb 16, 2024

Conversation

jakethekoenig
Copy link
Member

If leash mode is enabled the script is double checked with the leash_model before execution and a message is printed if its deemed unsafe.

A cute leash is added to the ascii art dog.

If leash mode is enabled the script is double checked with the
leash_model before execution and a message is printed if its deemed
unsafe.

A cute leash is added to the ascii art dog.
@biobootloader
Copy link
Member

Haha I'm scared to test this by asking for dangerous things

I think I'll modify temporarily to give me override after it decides so I can test

@jakethekoenig
Copy link
Member Author

Haha I'm scared to test this by asking for dangerous things

I think I'll modify temporarily to give me override after it decides so I can test

Good idea. I didn't really test this PR as much as I should

@biobootloader
Copy link
Member

So far it it's hard to find something the base model will do that the leash model won't let it!
image

@jakethekoenig
Copy link
Member Author

So far it it's hard to find something the base model will do that the leash model won't let it! image

Honestly I'm not really surprised. I added it because I knew Grant wanted it but I'm skeptical. In a lot of ways it seems like asking an llm the same thing twice. You're not really going to get new information. And if you really care about safety you're going to want to read the script first anyway.

@biobootloader
Copy link
Member

image

finally got one but the leash stops it and the reason is: it's safe 😆

@granawkins
Copy link
Member

So far it it's hard to find something the base model will do that the leash model won't let it! image

Honestly I'm not really surprised. I added it because I knew Grant wanted it but I'm skeptical. In a lot of ways it seems like asking an llm the same thing twice. You're not really going to get new information. And if you really care about safety you're going to want to read the script first anyway.

Ya I'm not married to this idea, it started out (like most things) as an offhand tweet. I enjoy using dry-run quite a lot, so maybe that's enough.

@jakethekoenig
Copy link
Member Author

So far it it's hard to find something the base model will do that the leash model won't let it! image

Honestly I'm not really surprised. I added it because I knew Grant wanted it but I'm skeptical. In a lot of ways it seems like asking an llm the same thing twice. You're not really going to get new information. And if you really care about safety you're going to want to read the script first anyway.

Ya I'm not married to this idea, it started out (like most things) as an offhand tweet. I enjoy using dry-run quite a lot, so maybe that's enough.

I definitely want at least the leash ASCII art from this pr

@granawkins granawkins mentioned this pull request Feb 16, 2024
15 tasks
@granawkins
Copy link
Member

I changed leash to just do what dry-run used to do.

@granawkins granawkins merged commit 911f05e into main Feb 16, 2024
2 checks passed
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.

None yet

3 participants