Add OpenLORIS to the Benchamarks about avalanche HOT 10 CLOSED

ok, cool! Please do not merge pull request directly into the master! Let me do that to make sure everything is in order! :-)

from avalanche.

will do it this weekend

from avalanche.

@ffeng1996 you can start by adding openloris as a pytorch dataset in avalanche/benchmarks/datasets.
You can take a look at how core50 is implemented there to have an idea.

from avalanche.

WIP PR #223 merged. It seems the validation.zip file is downloaded instead of the train. A this point it is better to let the user download it manually to remove any confusion.

@ffeng1996 can you please start creating the "continual" data loader? You can take inspiration from the CORe50 one here: avalanche\benchmarks\classic\core50.py. Thanks!

from avalanche.

WIP PR #223 merged. It seems the validation.zip file is download instead of the train. A this point it is better to let the user download it manually to remove any confusion.

@ffeng1996 can you please start creating the "continual" data loader? You can take inspiration from the CORe50 one here: avalanche\benchmarks\classic\core50.py. Thanks!

OK. I will write down some comments to let the user download the datasets manually and finish the continual setting. Thanks for taking the time.

from avalanche.

Hi @ffeng1996, I've already fixed it here: vlomonaco@a07ae71

You can implement directly the new module for the two continual learning benchmarks based on openloris (IROS & ICRA versions)!

from avalanche.

Hi @ffeng1996, I've already fixed it here: a07ae71

You can implement directly the new module for the two continual learning benchmarks based on openloris (IROS & ICRA versions)!

Got it. Many thanks for your help!

from avalanche.

I have submitted the continual data generator for openloris (https://github.com/vlomonaco/avalanche/pull/229). Many thanks for the continuous support and help.

from avalanche.

Thanks @ffeng1996 I've merged it in vlomonaco@67c68f6.

This looks good but we should support both the IROS and ICRA version I believe! Can you adapt the OpenLoris and OPENLORIS_DATA classes so that the user is able to choose which version to use? Thanks!

from avalanche.

Thanks @ffeng1996 I've merged it in 67c68f6.

This looks good but we should support both the IROS and ICRA version I believe! Can you adapt the OpenLoris and OPENLORIS_DATA classes so that the user is able to choose which version to use? Thanks!

Sure. I added the IROS benchmark together with the updated bacth_fileliss in #230. Thanks a lot!

from avalanche.