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

Initialize BaseOnlineStrategy #880

Merged
merged 4 commits into from Jan 21, 2022

Conversation

HamedHemati
Copy link
Collaborator

This PR initializes BaseOnlineStrategy. As discussed before, this is meant to be considered as a possible solution(and not necessarily the final soliton) for the implementation of online strategies. Three new files are added:

avalanche/training/strategies/base_online_strategy -> the base strategy for online strategies
avalanche/training/strategies/strategy_wrappers_online.py -> tried to avoid mixing strategy wrappers
examples/online_naive -> an example for testing

@coveralls
Copy link

coveralls commented Jan 18, 2022

Pull Request Test Coverage Report for Build 1726076854

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.01%) to 79.914%

Totals Coverage Status
Change from base Build 1707322059: 0.01%
Covered Lines: 11570
Relevant Lines: 14478

💛 - Coveralls

@AntonioCarta
Copy link
Collaborator

Thanks @HamedHemati!

As we discussed at the previous meeting, there are many issues with this approach that need to be addressed. I think we can accept this PR to use it as a starting point. I will open a separate issue to provide a roadmap for the next steps.

@AntonioCarta AntonioCarta merged commit c8d11a5 into ContinualAI:master Jan 21, 2022
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.

None yet

3 participants