ami-iit / element_human-action-intention-recognition

8 stars 0 forks source link

Cleanup the code for data class structure #19

Open kouroshD opened 4 years ago

kouroshD commented 4 years ago

Currently, in this repo, we have one data class, and all the implementations for different types of data generation or data reading are added to the same class. However, different problems needs different implementation for the data generation, reading, and plotting, and preparation to feed the RNN. In this repo, we follow the activities to clean the structure of the data class. We define an abstract base class, and some common implementation will be added to this base class. The specific implantation, related to each data type will be added to the inherited class.

yeshasvitirupachuri commented 4 years ago

@kouroshD is there any update on this issue ?

kouroshD commented 3 years ago

I did some part of this issue in commit https://github.com/dic-iit/element_human-action-intention-recognition/commit/b6d7e8c76bab6f7f8b436ad29dbd9ff62f424756 and https://github.com/dic-iit/element_human-action-intention-recognition/commit/38e0b8f0eba73dc22e77cd7e37f9794b926a954f

claudia-lat commented 3 years ago

Any update @kouroshD ? Otherwise we can close it.

kouroshD commented 3 years ago

No, update on this. I should address this issue during RI8 and make the code open source.

kouroshD commented 3 years ago

I've moved the issue to icebox and I change the relevant epic of the issue.

claudia-lat commented 3 years ago

@kouroshD be aware that you have to move the issue to the Icebox of ergoCub as well.