search

kevinhughes27 / TensorKart

self-driving MarioKart with TensorFlow
MIT License
1.58k stars 256 forks source link

Utils Refactor #32

Closed kevinhughes27 closed 7 years ago

kevinhughes27 commented 7 years ago

While testing @millerda's branch for #3 I made the following refactors:

remove prepare_image and clean up resize_image. I stepped through a full cycle to ensure everything still works

Next I split the Screenshot class into 2 classes adding a Sample class which represent the image after it has been transformed to be a sample.

I also removed the perfTests since they didn't actually use the utils there is no real reason to keep them around.

@bzier can you have a look as well

bzier commented 7 years ago

Hey, I've been out of town for the last week. I'll take a look at this stuff when I get back, or this weekend.

bzier commented 7 years ago

Hey Kevin, sorry I never got around to reviewing this. I've been really busy the last couple months and haven't spent any time on this project. I'd like to get back to it, but don't know how soon that might happen.

kevinhughes27 commented 7 years ago

No worries! If I broke something I'm sure someone will report it :)