Skip to content

code review - weixuan jiang #11

@jweixuan

Description

@jweixuan

In general, there are so many branches in your project and some files are repeated placed in different branches, which casues reviewer don't know your group division of labor cooperation.
Also, the README file under the master branch doesn't list android folder in Navigation Folders. The android app program in master branch seems to be nice. Your guys use COCO dataset from Microsoft and KITTI that is really great by exploring and using existing resources.
And, the result shown in your pictures meet your requirements. Nice works.

As for each sub-branch or personal branch, because of the problem of your chaotic branch, I can't easily disticguish which part of your project was implement by which groupmate. Maybe your guys can delete or remove some repeated files to make each one contributions more clearly later.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions