search

Wwwzff / Ship-Detection

0 stars 3 forks source link

Group 1 /wheelchair #7

Open wzezhong opened 6 years ago

wzezhong commented 6 years ago

I am very glad to review your code. Your code looks good, you have so many code to build your project. It is awesome. For your read me file is clearly now which helped me to understand every part for your work and what you have get or not. However, you can separate your code to many different parts because you whole code is built by many different function, it is very good for you and other people to read your code. You are very great to connect Professor Osama's mini project 2 with your project. It is an efficient way for your group. And it also help you save many time to do different research. I think that you can use more part about Tensorflow or try to different tools to finish your work. Because when you compare with two different tools, you will know which one is better for you. And you can void waste much time.

Wwwzff commented 6 years ago

Thanks for your review work my friends, and really sorry for delay of updating github which has been done in a hurry and also left us little time to make more detailed comments in our code. We'll sure seperate our main code into small parts and I know it's containing lots of functions and may cause you problems when reading it through. Actually what we're doing now is trying to use another kind of model to train our dataset, but the progress is a bit slow and thus we didn't count this part in in this repository. Maybe in the near future when the work's done we'd like to update it and feel free to raise suggestions or questions at that time. Also hope you go well with your wheelchair control system : )

wzezhong commented 6 years ago

Thank you for let me know about your project. You project is very good and I make sure you have a good job.

On Fri, Nov 2, 2018 at 4:40 PM Wwwzff notifications@github.com wrote:

Thanks for your review work my friends, and really sorry for delay of updating github which has been done in a hurry and also left us little time to make more detailed comments in our code. We'll sure seperate our main code into small parts and I know it's containing lots of functions and may cause you problems when reading it through. Actually what we're doing now is trying to use another kind of model to train our dataset, but the progress is a bit slow and thus we didn't count this part in in this repository. Maybe in the near future when the work's done we'd like to update it and feel free to raise suggestions or questions at that time. Also hope you go well with your wheelchair control system : )

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/Wwwzff/Ship-Detection/issues/7#issuecomment-435494706, or mute the thread https://github.com/notifications/unsubscribe-auth/ApBlUY3nN4N2AitVpN0T--R5iHDqbd4Uks5urK2agaJpZM4YMOkW .