Skip to content
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

Fix getMemoryReport ArrayIndexOutOfBoundsException #8015

Merged
merged 1 commit into from Jul 17, 2019

Conversation

@jxtps
Copy link
Contributor

commented Jul 16, 2019

Fix Upsampling.getMemoryReport ArrayIndexOutOfBoundsException, see #8014

Fix getMemoryReport ArrayIndexOutOfBoundsException
Fix Upsampling.getMemoryReport ArrayIndexOutOfBoundsException, see #8014
@AlexDBlack
Copy link
Contributor

left a comment

LGTM, thanks!

@AlexDBlack

This comment has been minimized.

Copy link
Contributor

commented Jul 17, 2019

Actually, before this can be merged: please read this
https://deeplearning4j.org/eclipse-contributors

@AlexDBlack

This comment has been minimized.

Copy link
Contributor

commented Jul 17, 2019

Hm, actually, this edit was done via github? The commit is definitely signed, that's a known issue on the Eclipse side for GitHub-based commits (as far as I know there's no fix yet).
ECA has been signed according to this: #8011 (comment)

So I'll merge this...

@AlexDBlack AlexDBlack merged commit 5a0904c into eclipse:master Jul 17, 2019

1 check failed

eclipsefdn/eca The author(s) of the pull request is not covered by necessary legal agreements in order to proceed.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.