-
Notifications
You must be signed in to change notification settings - Fork 80
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
Change brainio #963
base: master
Are you sure you want to change the base?
Change brainio #963
Conversation
…oralIgnore to Temporal Aligned
Update to the newest origin main.
merge new updates from origin main
Co-authored-by: Martin Schrimpf <[email protected]>
Co-authored-by: Martin Schrimpf <[email protected]>
Co-authored-by: Martin Schrimpf <[email protected]>
Co-authored-by: Martin Schrimpf <[email protected]>
The tests on github include the tests on models but only one benchmark test (alexnet on MajajHong2015). Therefore, here I add tests:
And the test report:
It seems all failed tests are due to private access permission, and I am not sure about the one skipped test. @mschrimpf Could you check if this is a good indication of ready to merge? |
indeed looks like failures are only due to private access, so I think this should be good. |
This is not a PR
This is just to test the new brainio. Don't merge this branch.