BiomedSciAI / fuse-med-ml

A python framework accelerating ML based discovery in the medical field by encouraging code reuse. Batteries included :)
Apache License 2.0
137 stars 34 forks source link

Make BackboneResnet3D backward compatible #217

Closed agiova closed 1 year ago

agiova commented 1 year ago

Describe the bug\ BackboneResnet3D expects the instance variable self.pool which was not present in models trained and stored with fuse-med-ml<0.2.8 See: https://github.com/BiomedSciAI/fuse-med-ml/blob/master/fuse/dl/models/backbones/backbone_resnet_3d.py#L83

FuseMedML version\ FuseMedML 0.2.8 used.

Python version\ 3.8.15

To reproduce\ Run forward pass with a model created with fuse-med-ml<0.2.8.

Expected behavior\ Models generated with previous versions should run.

agiova commented 1 year ago

Thank you @mosheraboh for merging!