Add conversion code to the BOP format #12
Conversation
…cene camera. Remove wrong dict key scene_camera
…r object without a texture map (solid color objects)
…script to extract model info from hot3d model info.
…n, skip Trimesh processing.
…re fully occluded in one of the streams
|
Hi @AnasIbrahim! Thank you for your pull request and welcome to our community. Action RequiredIn order to merge any pull request (code, docs, etc.), we require contributors to sign our Contributor License Agreement, and we don't seem to have one on file for you. ProcessIn order for us to review and merge your suggested changes, please sign at https://code.facebook.com/cla. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need to sign the corporate CLA. Once the CLA is signed, our tooling will perform checks and validations. Afterwards, the pull request will be tagged with If you have received this in error or have any questions, please contact us at [email protected]. Thanks! |
|
Thank you @AnasIbrahim,
|
facebook-github-bot
added
the
CLA Signed
|
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Meta Open Source project. Thanks! |
|
I added the description and signed the license. |
|
@SeaOtocinclus has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
| @@ -0,0 +1,329 @@ | |||
| """ | |||
There was a problem hiding this comment.
All files must have a header related to licensing of the project
# Copyright (c) Meta Platforms, Inc. and affiliates.
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
There was a problem hiding this comment.
I added the license. I also fixed an existing problem with the generation of out-of-frame-scope masks.
There was a problem hiding this comment.
Thank you @AnasIbrahim, I will fix the linting issue.
|
@SeaOtocinclus has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@SeaOtocinclus I just made a trivial change to the argparse input parameters. To make it easier to automate the conversion on the BOP server. |
|
@SeaOtocinclus has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
|
@AnasIbrahim Look good to me, I will merge. |
|
@SeaOtocinclus merged this pull request in 1d766a3. |
This PR contains 3 scripts to convert the HOT3D-Clips dataset from its native format to the BOP format. The BOP format had to be extended because the original format only included RGB and depth modalities. This extension made it possible to include a varying number of modalities (RGB, gray1, gray2) with separate annotations for each modality. The dataset has already been converted using these scripts and uploaded to the BOP website. These scripts went through several revisions, as the conversion process required multiple changes to adapt the dataset to the new standard format, which was initially not strictly defined and required frequent adjustments. The scripts in this PR are isolated and do not make any changes to the existing code in the repository.
The 3 Scripts are: