scil-vital / dwi_ml

Code for people working on machine learning for diffusion MRI
MIT License
8 stars 7 forks source link

Removing unnecessary params model_uses_forward and loss_uses_forward #218

Closed EmmaRenauld closed 11 months ago

EmmaRenauld commented 11 months ago

Now always sending streamlines both in forward and in loss. Much simpler, not heavier. User can decide to ignore.

EmmaRenauld commented 11 months ago

@arnaudbore As I told you a couple weeks ago: If this passes, after merging, you can remove your line self.forward_uses_streamlines = True in your model.