vchoutas / smplx

SMPL-X
https://smpl-x.is.tue.mpg.de/
Other
1.73k stars 301 forks source link

Why disable PCA for MANO when num_pca_comps == 45 #192

Open jianglongye opened 2 months ago

jianglongye commented 2 months ago

Thank you for this great repo! We've come across an issue in our usage.

Why is PCA disabled for MANO when the number of PCA components is set to 45?

Here is the relevant code: https://github.com/vchoutas/smplx/blob/1265df7ba545e8b00f72e7c557c766e15c71632f/smplx/body_models.py#L1599

This issue is problematic when using the PCA component parameter set to 45, such as with the DexYCB dataset.