Skip to content

Update new.toml#1

Open
yashasvigirdhar wants to merge 1 commit into
philschmid:mainfrom
yashasvigirdhar:patch-1
Open

Update new.toml#1
yashasvigirdhar wants to merge 1 commit into
philschmid:mainfrom
yashasvigirdhar:patch-1

Conversation

@yashasvigirdhar
Copy link
Copy Markdown

Remove an instruction from new.toml READ ONLY MANDATE as it doesn't belong there.

That bullet is for telling it what its not supposed to do.

Remove an instruction from new.toml READ ONLY MANDATE as it doesn't belong there.
@lirantal
Copy link
Copy Markdown

Hey Yash, it uses the tools to gather information, not for the purpose of writing anything to disk or otherwise. Did you experience any other behavior despite the model grounding for read-only mode?

@yashasvigirdhar
Copy link
Copy Markdown
Author

So basically, as I understood it, this list is for the things that might change the codebase and it shouldn't do that.

Giving it instruction to use the available tools is already present in ### 1. Investigation Phase below.

That's my understanding.

@lirantal
Copy link
Copy Markdown

How would it "understand the codebase" without access to tools like running "ls", "grep", "find", etc ?

@yashasvigirdhar
Copy link
Copy Markdown
Author

isn't understanding the codebase part of ### 1. Investigation Phase below? that has a bullet on using the available tools.

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