-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
fixes #15836; WIP, [backport:1.4] #16058
Conversation
@mratsim This makes arraymancer tests fail but it might be that the compiler now rightly complains about one of your tests cases. Please investigate. |
Looking in the error: https://github.com/nim-lang/Nim/pull/16058/checks?check_run_id=1430415082#step:11:1064 And it's location: https://github.com/mratsim/Arraymancer/blob/4ae9b81180ac7d6e5361baa9655f4f8d8100b074/tests/nn_primitives/test_nnp_convolution.nim#L153 It's a compile-time error and I think it's likely that sugar |
Ok, so in other words, you're confirming that your test cases are correct. Thanks, will investigate further. |
This pull request has been automatically marked as stale because it has not had recent activity. If you think it is still a valid PR, please rebase it on the latest devel; otherwise it will be closed. Thank you for your contributions. |
Refresh. |
This pull request is stale because it has been open for 1 year with no activity. Contribute more commits on the pull request and rebase it on the latest devel, or it will be closed in 30 days. Thank you for your contributions. |
This pull request has been marked as stale and closed due to inactivity after 395 days. |
No description provided.