Expose nclasses option for DenseNet#154
Expose nclasses option for DenseNet#154darsnack merged 2 commits intoFluxML:masterfrom theabhirath:densenet-classes
nclasses option for DenseNet#154Conversation
|
Instead of maintaining a separate non-breaking branch, maybe the easiest thing would be to revert the breaking conv + bn changes since they weren't a major impact to TTFG anyways. We can release the other changes along with this and keep the breaking stuff in a separate, new PR. |
I've opened #155 to address this. I will run some more tests when I get time to ascertain just how much returning a |
Previously, the `nclasses` option was not exposed for the `DenseNet` API with the `config` option.
|
Can you bump the patch too? |
Previously, the
nclassesoption was not exposed for theDenseNetAPI with theconfigoption.This PR adds that. Would be good to have a patch release too if possible (only with this commit and not the previous breaking change), since I'm trying to train some models and this will help with that 😅