-
Notifications
You must be signed in to change notification settings - Fork 44.6k
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
Fix path processing #5032
Fix path processing #5032
Conversation
✅ Deploy Preview for auto-gpt-docs canceled.
|
You changed AutoGPT's behaviour. The cassettes have been updated and will be merged to the submodule when this Pull Request gets merged. |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #5032 +/- ##
==========================================
+ Coverage 50.93% 51.07% +0.13%
==========================================
Files 119 119
Lines 4900 4973 +73
Branches 650 664 +14
==========================================
+ Hits 2496 2540 +44
- Misses 2219 2236 +17
- Partials 185 197 +12
☔ View full report in Codecov by Sentry. |
Background
We continue to have issues where a piece of code containing a relative path reference is moved and suddenly the
logs
folder or the workspace folder isn't in the right place anymore.Changes
logs/
back to project root after it was misplaced in Restructure logs.py into a module; include log_cycle #4921./auto_gpt_workspace
after it was misplaced in Create run_task python hook to interface with benchmarks #4778 (see this comment)Config.workspace_path
type toPath
Documentation
Test Plan
Auto-GPT runs fine, tests pass
PR Quality Checklist