-
Notifications
You must be signed in to change notification settings - Fork 17
Implement comprehensive logging system with configurable levels, file output, and WebSocket streaming #181
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
base: master
Are you sure you want to change the base?
Conversation
… support Co-authored-by: openuc2-gmbh <218447657+openuc2-gmbh@users.noreply.github.com>
Co-authored-by: openuc2-gmbh <218447657+openuc2-gmbh@users.noreply.github.com>
Co-authored-by: openuc2-gmbh <218447657+openuc2-gmbh@users.noreply.github.com>
Co-authored-by: openuc2-gmbh <218447657+openuc2-gmbh@users.noreply.github.com>
Fixed the circular import issue in commit The fix:
The logging system now initializes correctly after all dependencies are loaded. Fixed the circular import issue in commit The fix:
The logging system now initializes correctly after all dependencies are loaded. |
Co-authored-by: beniroquai <4345528+beniroquai@users.noreply.github.com>
@copilot I still get these errors:
|
... Could you provide the complete error message? The error trace appears to be cut off. The full stack trace will help me identify and fix the issue. You can paste it directly in the comment or attach the full log file. |
Implementation Plan for Logging System Enhancement
ImSwitchConfig
class (log level, log file settings)logging.py
to support configurable log levels from configLatest Changes (Commit: Fix circular import issue)
Fixed a critical circular import problem that was preventing ImSwitch from starting:
Problem
The logging setup was being called too early in the initialization process (before the framework was fully loaded), causing circular import dependencies:
__main__.py
tried to importsetup_logging
fromimswitch.imcommon.model
__init__.py
which imports framework componentsSolution
setup_logging()
call to AFTER the framework imports in__main__.py
from imswitch.imcommon.model import dirtools
to direct importfrom imswitch.imcommon.model.dirtools import UserFileDirs
~/ImSwitchConfig
if dirtools can't be importedChanges Made
imswitch/__main__.py
: Moved logging setup to after framework initializationimswitch/imcommon/model/logging.py
: Fixed dirtools imports with try/except fallbackThis ensures logging is initialized at the right time in the startup sequence and prevents circular import issues.
Original prompt
Fixes #180
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.