Skip to content
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

Long term perspectives #84

Open
ElliotMaitre opened this issue Sep 26, 2023 · 0 comments
Open

Long term perspectives #84

ElliotMaitre opened this issue Sep 26, 2023 · 0 comments

Comments

@ElliotMaitre
Copy link
Collaborator

ElliotMaitre commented Sep 26, 2023

No particular order in the list

  1. Standardizing the code structure accross MegaPose and Cosypose, in particuliar:
  • API Detector (Done)
  • [API Coarse
  • API Refiner
    —> Standardizing the two approaches would then make it possible to create a more modular architecture for integrating other methods.
  1. Decoupling detection / pose estimation. Currently prediction_runner and pose_estimator handle this... Decoupling would allow greater flexibility.

  2. in megapose/model/pose_rigid, move PosePredictorOutputCosypose elsewhere, but where?

  3. More generally, some code is duplicated between MegaPose and CosyPose (pose_predictor.py, ...). A refactoring of the code to make a general API for HappyPose would be needed. For now, it is similar in both repositories but some work is still needed to make something really clean.

  4. Currently, there are pointers between pose_estimators. For example, CosyPose use the evaluation of MegaPose, which is stored in pose_estimators.megapose. This should be looked at, as a pose_estimator should probably import only from itself and the toolbox part.

  5. If re-training CosyPose (if necessary?) Adapt object names to choices made in MegPose (change from obj-000001 to ycbv_obj-000001, for example, in the config of each model).

  6. For CosyPose training, pose_training.py is working on a single GPU but it seems that it doesn't work on Jean Zay. More investigation on why is needed. detector_training.py shouldn't be working yet, as no one worked on it.

  7. Rewriting of the repo using more recent technologies (e.g. PytorchLightning, Weight&Biases)

  8. Update of the Neural Backbone (Yolo for detection ?, PoseEstimator ?)

  9. A good look over the code to remove potentially useless elements (many (unused?) scripts ...) -> Compare with entry points and uses cases?

  10. Docker/singularity/apptainer for Happypose ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant