Skip to content

Added IL2M strategy#1682

Merged
AntonioCarta merged 2 commits intoContinualAI:masterfrom
iretes:IL2M
Feb 21, 2025
Merged

Added IL2M strategy#1682
AntonioCarta merged 2 commits intoContinualAI:masterfrom
iretes:IL2M

Conversation

@iretes
Copy link
Copy Markdown
Contributor

@iretes iretes commented Feb 19, 2025

Implementation of the class-incremental method "Incremental Learning with Dual Memory" (IL2M).

@coveralls
Copy link
Copy Markdown

coveralls commented Feb 19, 2025

Pull Request Test Coverage Report for Build 13441318849

Details

  • 90 of 101 (89.11%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.1%) to 51.118%

Changes Missing Coverage Covered Lines Changed/Added Lines %
avalanche/training/supervised/strategy_wrappers.py 7 8 87.5%
tests/training/test_strategies.py 1 5 20.0%
avalanche/training/plugins/il2m.py 81 87 93.1%
Totals Coverage Status
Change from base Build 11572207189: 0.1%
Covered Lines: 14726
Relevant Lines: 28808

💛 - Coveralls

@AntonioCarta
Copy link
Copy Markdown
Collaborator

Hi @iretes, thanks again for your contribution. Everything looks fine. You just need to run black to change the code style.
Can you run the command black . and update the PR?

https://black.readthedocs.io/en/stable/index.html

@iretes
Copy link
Copy Markdown
Contributor Author

iretes commented Feb 20, 2025

Thanks for your feedback, @AntonioCarta. I had already run black ., and I believe the issue was actually due to missing imports for quoted types. I've just fixed it.

@AntonioCarta AntonioCarta merged commit 5a6a018 into ContinualAI:master Feb 21, 2025
@AntonioCarta
Copy link
Copy Markdown
Collaborator

Thanks again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants