unibas-gravis / scalismo

Scalable Image Analysis and Shape Modelling
https://scalismo.org
Apache License 2.0
245 stars 65 forks source link

adding attributes to the shape model #228

Closed Ghazi-Bouabene closed 2 years ago

Ghazi-Bouabene commented 6 years ago

It would be useful if one could combine some additional attributes, such as the subject's age, weight, height, gender, or even some properties of the shape itself, e.g. length or width of the bone, together with the shape model.

An example use case of such a combined model would be to complete a fractured bone by conditioning the model, not only on the observed healthy part, but additionally on the patient's attributes: age, weight, length of the contralateral bone , etc., which should improve the completion results.

The method presented in this paper https://link.springer.com/chapter/10.1007/978-3-319-64870-5_5 could for example be used in the implementation (generalized to all shapes, not just faces).

BernhardEgger commented 6 years ago

I like the idea to include such a method and would be willing to contribute. Our work (link above) presents the model and includes an evaluation using model metrics. It does not include an analysis application besides attribute prediction. Besides the implementation, I think there should be an example application/paper that proves the usefulness in an analysis setting. Compared to other methods to include attributes, the proposed one can also include categorical data which is often available in a clinical setting. There seems to be a demand in the community and a range of applications:

marcelluethi commented 6 years ago

This sounds interesting, indeed. I have some experience in this topic with statismo, where we integrated the approach of Remi Blanc . There the method was easy to implement and nicely fit into the architecture. What was a bit more difficult was the data management. I suppose, information about the attributes (name, etc.) would need to be managed and there should also be the possibility to make everything persistent. I don't know how much overhead this will add to Scalismo?

BernhardEgger commented 6 years ago

I don't know how well the method of Remi Blanc fits into Scalismo. For the copula method, it is definitely possible to squeeze it in. It, however, breaks with some ideas the framework was originally built on - the dependency structure is built on values that have no more a "meaning". The overhead in Scalismo is more a decision that should be done on a high-level. The alternative would be to provide the implementation as a separate project. Since the applicability of both methods is quite wide, I would argue to put it as part of the Scalismo framework.

marcelluethi commented 2 years ago

As this issue has been inactive for a long time, I am closing it. In case there is still interest, we can open a new issue once we start with the implementation.