roboticslab-uc3m / vision

Vision processing
https://robots.uc3m.es/vision/
11 stars 7 forks source link

New name for cv1 -> colorRegionDetection #22

Closed jgvictores closed 8 years ago

jgvictores commented 8 years ago

For starters, "1" should be dropped. It's mainly composed by: classical morphological operations, some kind of binary thresholding, then opencv blobs/contours... @David-Estevez Any ideas?

David-Estevez commented 8 years ago

What about "cvUtils" or "cvTools"?

Although the prefix "cv" gives me the impression that the code belongs to the OpenCV main code... "VisionUtils" or "VisionTools" also sound good to me.

jgvictores commented 8 years ago

+1 to cv -> vision Utils or Tools sounds more like collections of scripts, how about some of the following or variants of them? "visionColorBlobs" "visionColorSegmentation"

David-Estevez commented 8 years ago

how about "BasicVision" or "Vision"?

jgvictores commented 8 years ago

I've always liked lowerCamelCase for programs, and category as prefix (even in contrast with standard english :smile:). Ok with "visionBasic"?

This change will not be immediate, but it's good as we converge into a naming standard.

David-Estevez commented 8 years ago

I thought it was related to some sort of library, so I used CamelCase for it.

Just as a side-note, computer vision literature (or at least the book I have at hand) names the different stages in vision using the follow naming:

jgvictores commented 8 years ago

Final decision:

jgvictores commented 8 years ago

Done at https://github.com/roboticslab-uc3m/teo-head/commit/dbf20fec53ba9699558c06adf9c4ff25472f7c0f.