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 TOGETHER_API_KEY to keys.cfg section of README #34

Merged
merged 1 commit into from
Apr 4, 2024

Conversation

0xdevalias
Copy link
Contributor

@0xdevalias 0xdevalias commented Apr 4, 2024

I noticed there is also a TOGETHER_API_KEY key that can be set in keys.cfg while I was investigating #26 (comment), but it wasn't mentioned in the README, so I wanted to add it:

# Set Together key
cfg = config.Config(os.path.join(os.getcwd(), "keys.cfg"))
together.api_key = cfg.TOGETHER_API_KEY

I also moved the GITHUB_TOKEN to the top of the list, since it's a required token.

I noticed there is also a `TOGETHER_API_KEY` key that can be set in `keys.cfg`, but it wasn't mentioned in the README, so wanted to add it:

https://github.com/princeton-nlp/SWE-agent/blob/6c9ebf0ea8a263806b276da7ba3b1eda1f4a9475/sweagent/agent/models.py#L509-L511
Copy link
Member

@klieret klieret left a comment

Choose a reason for hiding this comment

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

Thanks!

@klieret klieret merged commit 7903883 into princeton-nlp:main Apr 4, 2024
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