uwsbel / autonomy-research-testbed

https://projects.sbel.org/autonomy-toolkit
MIT License
16 stars 6 forks source link

Imitation Learning ML training needs to be added somewhere #39

Open StefanCaldararu opened 1 year ago

StefanCaldararu commented 1 year ago

@harryzhang1018 and I have a seperate repo where we trained the Keras files for the HIL imitation learning models. This should live somewhere related to this repo, but:

here is a link to the current repo: https://github.com/harryzhang1018/ML_Control

@AaronYoung5 any suggestions?

AaronYoung5 commented 1 year ago

Can it live here?

And does it need to be a different repo? Is it used for other applications? It could also go at the top level of this repo in a subfolder called utilties or something

AaronYoung5 commented 1 year ago

@StefanCaldararu @harryzhang1018 still relevant? What's the thoughts here? Ig i wasn't part of any conversations relating to this

StefanCaldararu commented 1 year ago

@StefanCaldararu @harryzhang1018 still relevant? What's the thoughts here? Ig i wasn't part of any conversations relating to this

Yes I think it is still relevant. IMO should be somewhat separated from this repo, but doesn't have to be completely separate. Utilities might be a good solution. Sorry I haven't had much time for this cleanup related stuff lately, maybe @harryzhang1018 can help out more? :)

harryzhang1018 commented 1 year ago

@StefanCaldararu @harryzhang1018 still relevant? What's the thoughts here? Ig i wasn't part of any conversations relating to this

Yes I think it is still relevant. IMO should be somewhat separated from this repo, but doesn't have to be completely separate. Utilities might be a good solution. Sorry I haven't had much time for this cleanup related stuff lately, maybe @harryzhang1018 can help out more? :)

Yes, I can jump in for this. Currently the Imitation Learning (IL) repo is still in a mess. Definitely need to do clean up.

Another thoughts: for icra, we did lots of outdoor testing, which data has been stored currently in this ML_Control repo. Any thoughts where should we put these data files?

StefanCaldararu commented 1 year ago

@StefanCaldararu @harryzhang1018 still relevant? What's the thoughts here? Ig i wasn't part of any conversations relating to this

Yes I think it is still relevant. IMO should be somewhat separated from this repo, but doesn't have to be completely separate. Utilities might be a good solution. Sorry I haven't had much time for this cleanup related stuff lately, maybe @harryzhang1018 can help out more? :)

Yes, I can jump in for this. Currently the Imitation Learning (IL) repo is still in a mess. Definitely need to do clean up.

Another thoughts: for icra, we did lots of outdoor testing, which data has been stored currently in this ML_Control repo. Any thoughts where should we put these data files?

Some sort of webpage would be nice to be able to display past results relating to ART. bunch of videos, datasets, and maybe even images of the containers so we can go back to past result states easily? IDK that's probably a lot of work but would be cool...

harryzhang1018 commented 1 year ago

@StefanCaldararu @harryzhang1018 still relevant? What's the thoughts here? Ig i wasn't part of any conversations relating to this

Yes I think it is still relevant. IMO should be somewhat separated from this repo, but doesn't have to be completely separate. Utilities might be a good solution. Sorry I haven't had much time for this cleanup related stuff lately, maybe @harryzhang1018 can help out more? :)

Yes, I can jump in for this. Currently the Imitation Learning (IL) repo is still in a mess. Definitely need to do clean up. Another thoughts: for icra, we did lots of outdoor testing, which data has been stored currently in this ML_Control repo. Any thoughts where should we put these data files?

Some sort of webpage would be nice to be able to display past results relating to ART. bunch of videos, datasets, and maybe even images of the containers so we can go back to past result states easily? IDK that's probably a lot of work but would be cool...

like this idea. Videos, datasets are easy. But idk about images, cuz we kinda lost track on this for long time and did twist inside container. But, I think it's important to keep in mind for this, for example, after outdoor convoy project, we should work with Thomas to create a image specific for this job.

AaronYoung5 commented 1 year ago

Shouldn't that go in public-metadata or chrono-reproducibility or whatever that repo is called? I agree docker is the way to go for reproducibility, but I don't think that would be a good place to store all data for projects. Do we need to keep around all the data? Will we ever reuse?

harryzhang1018 commented 1 year ago

Shouldn't that go in public-metadata or chrono-reproducibility or whatever that repo is called? I agree docker is the way to go for reproducibility, but I don't think that would be a good place to store all data for projects. Do we need to keep around all the data? Will we ever reuse?

Yea, I think for pic and results, maybe create separate projects on ART website might be helpful and cool.

StefanCaldararu commented 1 year ago

Shouldn't that go in public-metadata or chrono-reproducibility or whatever that repo is called? I agree docker is the way to go for reproducibility, but I don't think that would be a good place to store all data for projects. Do we need to keep around all the data? Will we ever reuse?

agreed. Not all data needs to stay, but we have a dataset that we trained on we should DEFINITELY keep that somewhere and not lose it

AaronYoung5 commented 1 year ago

good to bring this up with Dan/others to see thoughts. I can also see making a projects page under this url (like https://projects.sbel.org/autonomy-toolkit) but with public-metadata-like stuff (like a webpage for each paper we write with embedded images, links to datasets, docker image pull links, code, etc.).

Either way, github isn't probably the best place to store datasets. We an always link to it in readmes, but probs wanna put in box or drive and link.