search

KevinMenden / scaden

Deep Learning based cell composition analysis with Scaden.
https://scaden.readthedocs.io
MIT License
71 stars 25 forks source link

M256 drop out rates in M1024 #88

Closed khkk378 closed 1 year ago

khkk378 commented 3 years ago

Hi!

You're using the M256 DO rates in M1024. Is this on purpose?

cdn1024 = Scaden(
        model_dir=model_dir + "/m1024",
        model_name="m1024",
        seed=seed,
        hidden_units=M1024_HIDDEN_UNITS,
        do_rates=M256_DO_RATES,
    )
KevinMenden commented 3 years ago

Hi Rasmus,

thanks for reporting! No that's not on purpose and a bug :) I refactored this part of the code recently, where I must have introduced this :facepalm:

Fortunately it doesn't matter, as this is the model used for prediction and dropout is deactivated during prediction. Nevertheless, not good. Will fix it!

KevinMenden commented 3 years ago

Will probably get out a new release this evening because that's rather annoying.