This repository has been archived by the owner on Oct 31, 2023. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi, thanks for sharing the code of such a good work. This PR mainly refines codes related to Body UV head network by formatting the code in the canonical style and fixes some minor bugs as shown in related commits. An additional function of visualizing training metrics in a tensorboard is added which is compatible with current framework and easy to be enabled by a simple command-line argument.
Major modifications include:
Fix a typo in config.py;
Refine code style in vis.py and add more comments for understanding more easily;
Refine densepose evaluation in densepose_cocoeval.py by simplifying the extraction of geodesic distance between two vertices from the pre-computed matrix in a compact representaion. Code style is also refined;
Fix broken links in INSTALL.md and README.md (similar PR in Fix broken link to detectron in Install.md #211);
Refine the implementation of building body_uv_rcnn head network before the real training process starts. Specifically,
BODY_UV_RCNN.NUM_SEMANTIC_PARTS = 14
in config.py to represent the number of semantic body parts which is used to buildAnnIndex_lowres
layer;