Skip to content

Commit 28beb3f

Browse files
authored
Fix: Use correct cache directory for file logging (#1564)
## Describe your changes ## Checklist before requesting a review - [ ] Add unit tests for this change. - [ ] Make sure all tests can pass. - [ ] Update documents if necessary. - [x] Lint and apply fixes to your code by running `lintrunner -a` - [ ] Is this a user-facing change? If yes, give a description of this change to be included in the release notes. - [ ] Is this PR including examples changes? If yes, please remember to update [example documentation](https://github.com/microsoft/Olive/blob/main/docs/source/examples.md) in a follow-up PR. ## (Optional) Issue link
1 parent ab72d69 commit 28beb3f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

olive/engine/engine.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ def __init__(
9494
def initialize(self, log_to_file: bool = False, log_severity_level: int = 1):
9595
"""Initialize engine state. This should be done before running the registered passes."""
9696
if log_to_file:
97-
enable_filelog(log_severity_level, self.cache.cache_dir, self.workflow_id)
97+
enable_filelog(log_severity_level, self.cache.dirs.cache_dir, self.workflow_id)
9898

9999
# set cache dir environment variables
100100
# might be used by other parts of olive to cache data

0 commit comments

Comments
 (0)