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

More robust log dir path #1312

Merged

Conversation

mikekelly
Copy link
Contributor

@mikekelly mikekelly commented Apr 14, 2023

Background

When run from the root of the project python scripts/main.py the logger was logging in a directory outside of the project directory (in the parent)

Changes

Make the log directory path (../logs) relative to the directory the logger file is in (ie. ./scripts/). This puts the logs dir in the project directory.

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

@ccqpein ccqpein mentioned this pull request Apr 14, 2023
5 tasks
Copy link

@joshmarsden1994 joshmarsden1994 left a comment

Choose a reason for hiding this comment

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

👌

@richbeales richbeales merged commit d5423fd into Significant-Gravitas:master Apr 14, 2023
@mikekelly mikekelly deleted the more-robust-log-dir-path branch April 14, 2023 16:14
@nponeccop nponeccop mentioned this pull request Apr 14, 2023
1 task
sindlinger pushed a commit to Orgsindlinger/Auto-GPT-WebUI that referenced this pull request Sep 25, 2024
…st-log-dir-path

More robust log dir path
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.

4 participants