Skip to content

Helicone #457

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.

Sign up for GitHub

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

Jump to bottom
Open
waynehamadi wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Helicone #457

waynehamadi wants to merge 2 commits into from

Conversation

@waynehamadi
Copy link
Owner

@waynehamadi waynehamadi commented Jul 18, 2023

Background

Changes

Documentation

Test Plan

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes.
  • I have run the following commands against my code to ensure it passes our linters: 
    black .
isort .
mypy
autoflake --remove-all-unused-imports --recursive --ignore-init-module-imports --ignore-pass-after-docstring autogpt tests --in-place

sohrabsaran and others added 2 commits July 18, 2023 13:21
@sohrabsaran @Pwuts @waynehamadi
Fix execute_python_file workspace mount & Windows path formatting (S…
fb4a1d3 
…ignificant-Gravitas#4996)

* fix for Significant-Gravitas#4975

* Add TODO based on code comment.

* Use builtin `Path.as_posix()`

* Remove TODO

---------

Co-authored-by: Reinier van der Leer <[email protected]>
Signed-off-by: Merwane Hamadi <[email protected]>
@waynehamadi
Push reports to google drive
04d53e9 
Signed-off-by: Merwane Hamadi <[email protected]>
@github-actions
Copy link

github-actions bot commented Sep 15, 2023

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

conflicts size/m

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@waynehamadi @sohrabsaran