You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@systemml.apache.org by de...@apache.org on 2017/07/20 06:27:43 UTC
systemml git commit: [SYSTEMML-1774] Bugfix for
ConvolutionUtils#binaryOperationInPlace
Repository: systemml
Updated Branches:
refs/heads/master 6618a8516 -> 614f5ab52
[SYSTEMML-1774] Bugfix for ConvolutionUtils#binaryOperationInPlace
When using Arrays.fill() to fill the values of the destination 1-dimension
array by fromIndex and toIndex, the column index cix for the sparse matrix
block needs to be added into fromIndex and endIndex first.
Closes #584.
Project: http://git-wip-us.apache.org/repos/asf/systemml/repo
Commit: http://git-wip-us.apache.org/repos/asf/systemml/commit/614f5ab5
Tree: http://git-wip-us.apache.org/repos/asf/systemml/tree/614f5ab5
Diff: http://git-wip-us.apache.org/repos/asf/systemml/diff/614f5ab5
Branch: refs/heads/master
Commit: 614f5ab5287337daaf53d9598c3df04a7cbe12f0
Parents: 6618a85
Author: Fei Hu <hu...@gmail.com>
Authored: Wed Jul 19 23:18:51 2017 -0700
Committer: Deron Eriksson <de...@apache.org>
Committed: Wed Jul 19 23:18:51 2017 -0700
----------------------------------------------------------------------
src/main/java/org/apache/sysml/runtime/util/ConvolutionUtils.java | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/systemml/blob/614f5ab5/src/main/java/org/apache/sysml/runtime/util/ConvolutionUtils.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/sysml/runtime/util/ConvolutionUtils.java b/src/main/java/org/apache/sysml/runtime/util/ConvolutionUtils.java
index 76f4c34..07da5bc 100644
--- a/src/main/java/org/apache/sysml/runtime/util/ConvolutionUtils.java
+++ b/src/main/java/org/apache/sysml/runtime/util/ConvolutionUtils.java
@@ -92,7 +92,7 @@ public class ConvolutionUtils {
int prevDestIndex = 0;
for(int j = apos; j < apos+alen; j++) {
// Multiplication by zero. Assumption: aix is sorted.
- Arrays.fill(dest, cix+prevDestIndex, aix[j], 0);
+ Arrays.fill(dest, cix+prevDestIndex, cix+aix[j], 0);
prevDestIndex = aix[j]+1;
dest[ cix+aix[j] ] *= avals[j];
}