Skip to content
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

Complete implementation of new caching system, and extend #222

Open
fkuehlein opened this issue May 17, 2024 · 0 comments · Fixed by #224 or #236
Open

Complete implementation of new caching system, and extend #222

fkuehlein opened this issue May 17, 2024 · 0 comments · Fixed by #224 or #236
Labels
enhancement maintenance something should be improved or is outdated

Comments

@fkuehlein
Copy link
Collaborator

fkuehlein commented May 17, 2024

New caching was introduced with 93ab748 and merged with #219. See commit message for description and hints on how to proceed.

As a first step towards this: (EDIT: resolved as of #236)

The pylint check 'no-member' is currently disabled globally. Working on re-enabling it on 196-review_linting, but would trigger warnings in TestCached as well as RecurrenceNetwork and subclasses. In the latter cases, the old caching is still in place (see above linked commit). Should be reviewed and tested for.

@fkuehlein fkuehlein added enhancement maintenance something should be improved or is outdated labels May 17, 2024
@fkuehlein fkuehlein linked a pull request May 21, 2024 that will close this issue
@fkuehlein fkuehlein reopened this Aug 9, 2024
@fkuehlein fkuehlein linked a pull request Aug 13, 2024 that will close this issue
@fkuehlein fkuehlein reopened this Sep 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement maintenance something should be improved or is outdated
Projects
None yet
1 participant