-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
拡張KITTI形式 から Annofab点群形式(KITTIベース) にリネームする #110
Conversation
@@ -1,6 +1,6 @@ | |||
# annofab-3dpc-editor-cli | |||
|
|||
* [コマンドサンプル](user_guide/command_sample.md) | |||
* [KITTI 3D object detection](user_guide/kitti_3d_object_detection.md) | |||
* [拡張KITTI形式](user_guide/kitti_extension.md) | |||
* [KITTI形式](user_guide/kitti_3d_object_detection.md) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
リネームではないが、「Annofab点群形式」に表記を揃えるために変更した。
anno3d/app.py
Outdated
拡張kitti形式のファイル群をAnnoFabにアップロードします | ||
Annofab点群形式のファイル群をAnnoFabにアップロードします |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
通常は「Annofab点群形式」、見出しでは「Annofab点群形式(KITIIベース)」にした。
って話ですけど、個人的には、ほぼすべてAnnofab点群形式(KITIIベース)
とすべきに思います。
(KITIIベース)
という注釈は、KITTIがベースで無いAnnofab点群形式が存在し得て、且つそういうものが登場したときに紛れがなくなるように、意図的に付けているはずです。
ここで、取り除いたものを書いてしまうと、紛れが生まれてしまうので、Annofab点群形式(KITIIベース)
とタイトルがついているところの下部構造部分ならともかく、そうでないところはすべてAnnofab点群形式(KITIIベース)
にすべきに思います。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
対応しました。
(KITIIベース)という注釈は、KITTIがベースで無いAnnofab点群形式が存在し得て、且つそういうものが登場したときに紛れがなくなるように、意図的に付けているはずです。
PR出した時点では、KITTIベースでないAnnofab点群形式が存在した時点で、Annofab点群形式(KITIIベース)
に置換すればよいかなと考えていました。
が、今置換しても問題はないので、対応しました。
close #109
対応箇所
補足
通常は「Annofab点群形式」、見出しでは「Annofab点群形式(KITIIベース)」にした。