chainer / chainercv

ChainerCV: a Library for Deep Learning in Computer Vision
MIT License
1.48k stars 305 forks source link

Keyword consistency #159

Open Hakuyume opened 7 years ago

Hakuyume commented 7 years ago
Hakuyume commented 7 years ago

I think it is better to unify keywords. I guessed current rules from the codes.

Hakuyume commented 7 years ago

@yuyu2172 How do think of it? Do you know other implicit rules?

yuyu2172 commented 7 years ago

Thanks for summarizing this. I have no complaint for everything, but the second one. I think we were discussing on the usage of cls inside eval_detection. I think this is under discussion.

Other than that, here are some that I came up.

Hakuyume commented 7 years ago

@yuyu2172 thank you. I merged the rules you suggested. I changed the state of usage of class to "under discussion".

yuyu2172 commented 7 years ago

Copied from #163, these are rules for bounding boxes.

For images, a similar rule applies like below.

Hakuyume commented 7 years ago

@yuyu2172 Thank you. I merged them.

yuyu2172 commented 7 years ago

I think there is another rule.

yuyu2172 commented 6 years ago

I think this naming conventions should be included in the documentation.

Here are some rules I found used in ChainerCV.

Rules specific to instance segmentation should also be documented.

Hakuyume commented 6 years ago

I think this naming conventions should be included in the documentation.

:+1:

yuyu2172 commented 6 years ago

Another rule