-
Notifications
You must be signed in to change notification settings - Fork 697
[MNT] Clearly define the MLP as a class/nn.model #1864
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
In the nhits paper: https://arxiv.org/abs/2201.12886, shows that the MLP as component of the Block, the current code does integrates this already as series of appended layers. This pull request instead clearly defines the MLP as a class/nn.model.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Though, there is an error in the code if you execute it, constructor call is misnamed.
seems like there are still some failures - are these caused by your changes? |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #1864 +/- ##
=======================================
Coverage ? 85.74%
=======================================
Files ? 68
Lines ? 6594
Branches ? 0
=======================================
Hits ? 5654
Misses ? 940
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good now!
In the nhits paper: https://arxiv.org/abs/2201.12886, shows that the MLP as component of the Block, the current code does integrates this already as series of appended layers.
This pull request instead clearly defines the MLP as a class/nn.model.
Description
This PR ...
Checklist
pre-commit install
.To run hooks independent of commit, execute
pre-commit run --all-files
Make sure to have fun coding!