Closed ibro45 closed 4 months ago
The recent updates enhance the lighter
framework by replacing the directory
parameter with a more versatile path
parameter across several classes, offering flexibility in specifying either file or directory paths. This change necessitates corresponding updates to the initialization and setup logic. Additionally, the framework's handling of state dictionaries and logging metrics has been refined to improve functionality and maintainability.
Files | Change Summary |
---|---|
lighter/callbacks/writer/base.py and lighter/callbacks/writer/table.py |
Replaced directory with path in constructors and initialization logic for more flexible path specification. Updated methods to use self.path instead of deprecated variables. |
lighter/system.py |
Removed a type check for Metric instances before logging metrics in the _log_stats function. |
lighter/utils/model.py |
Revised logic for checking overlap between checkpoint and model state_dicts, including enhancing error messages with more key details. |
In paths and logs, we find our way, With more freedom, less dismay. Refined the checks, and keys relay, To keep our system in bright array.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Description
Time for v0.0.3 https://github.com/project-lighter/lighter/issues/118
Related Issue
https://github.com/project-lighter/lighter/issues/125 https://github.com/project-lighter/lighter/issues/121 https://github.com/project-lighter/lighter/issues/90
Type of Change
Checklist
CODE_OF_CONDUCT.md
document.CONTRIBUTING.md
guide.make codestyle
.Summary by CodeRabbit
New Features
Bug Fixes
Refactor
Improvements