-
Notifications
You must be signed in to change notification settings - Fork 6
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/remove unused cache leftovers #11
Conversation
the json dicts being saved are not used anywhere, just leftovers from the earlier experiment caching results
WalkthroughThe changes involve significant modifications to the Changes
Poem
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? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- init.py (4 hunks)
Additional comments not posted (5)
__init__.py (5)
Line range hint
15-23
: LGTM!The method is correctly implemented.
The code changes are approved.
Line range hint
25-26
: LGTM!The method is correctly implemented.
The code changes are approved.
Tools
Ruff
2-2:
typing.List
imported but unused(F401)
4-4:
ovos_utils.timed_lru_cache
imported but unused(F401)
Line range hint
28-39
: LGTM!The method is correctly implemented.
The code changes are approved.
Tools
Ruff
2-2:
typing.List
imported but unused(F401)
4-4:
ovos_utils.timed_lru_cache
imported but unused(F401)
Line range hint
10-13
: Verify the impact of removingJsonStorageXDG
instances.The removal of
JsonStorageXDG
instances forarchive
andplaylists
suggests a change in data management. Ensure that this change does not affect the class's functionality.Run the following script to verify if there are any remaining references to
self.archive
orself.playlists
in the codebase:Verification successful
No Remaining References to
self.archive
orself.playlists
FoundThe removal of
JsonStorageXDG
instances forarchive
andplaylists
does not affect the class's functionality, as there are no remaining references to these attributes in the codebase.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify if there are any remaining references to `self.archive` or `self.playlists`. # Test: Search for references to `self.archive` and `self.playlists`. Expect: No occurrences. rg --type python 'self\.archive|self\.playlists'Length of output: 81
Script:
#!/bin/bash # Description: Verify if there are any remaining references to `self.archive` or `self.playlists`. # Test: Search for references to `self.archive` and `self.playlists`. Expect: No occurrences. rg 'self\.archive|self\.playlists' --glob '*.py'Length of output: 50
Line range hint
41-91
: Verify the impact of removing storage logic.The removal of storage logic for playlist entries and their associated streams suggests a change in data management. Ensure that this change does not affect the method's functionality.
Run the following script to verify if there are any remaining references to
store
orself.archive
in the codebase:Verification successful
Verification Successful: No Impact from Removed Storage Logic
The removal of storage logic for playlist entries and their associated streams does not affect the method's functionality, as there are no remaining references to
store
orself.archive
in the codebase.Scripts executed
The following scripts were executed for the analysis:
Script:
#!/bin/bash # Description: Verify if there are any remaining references to `store` or `self.archive`. # Test: Search for references to `store` and `self.archive`. Expect: No occurrences. rg --type python 'store|self\.archive'Length of output: 71
Script:
#!/bin/bash # Description: Verify if there are any remaining references to `store` or `self.archive`. # Test: Search for references to `store` and `self.archive`. Expect: No occurrences. rg 'store|self\.archive' --glob '*.py'Length of output: 40
Tools
Ruff
2-2:
typing.List
imported but unused(F401)
4-4:
ovos_utils.timed_lru_cache
imported but unused(F401)
the json dicts being saved are not used anywhere, just leftovers from the earlier experiment caching results
Summary by CodeRabbit
New Features
Bug Fixes