ML4GLand / EUGENe

Elucidating the Utility of Genomic Elements with Neural Nets
MIT License
61 stars 4 forks source link

Dropout rate naming inconsistency #6

Closed haydenstites closed 1 year ago

haydenstites commented 2 years ago

In the BuildFullyConnected function used in the BasicFullyConnectedModule module the argument for dropout rate is dropout_rate, where in BasicConv1D it is named dropout_rates.

adamklie commented 2 years ago

I guess that's because the FCN only allows a single dropout rate across layers, whereas the conv allows different dropout rates for each layer. But we can allow the FCN to be more flexible too