/*
 * Licensed to the Apache Software Foundation (ASF) under one
 * or more contributor license agreements.  See the NOTICE file
 * distributed with this work for additional information
 * regarding copyright ownership.  The ASF licenses this file
 * to you under the Apache License, Version 2.0 (the
 * "License"); you may not use this file except in compliance
 * with the License.  You may obtain a copy of the License at
 * 
 *   http://www.apache.org/licenses/LICENSE-2.0
 * 
 * Unless required by applicable law or agreed to in writing,
 * software distributed under the License is distributed on an
 * "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
 * KIND, either express or implied.  See the License for the
 * specific language governing permissions and limitations
 * under the License.
 */


package org.apache.sysml.runtime.matrix.operators;

import org.apache.sysml.runtime.DMLRuntimeException;
import org.apache.sysml.runtime.functionobjects.Divide;
import org.apache.sysml.runtime.functionobjects.GreaterThan;
import org.apache.sysml.runtime.functionobjects.GreaterThanEquals;
import org.apache.sysml.runtime.functionobjects.LessThan;
import org.apache.sysml.runtime.functionobjects.LessThanEquals;
import org.apache.sysml.runtime.functionobjects.Power;
import org.apache.sysml.runtime.functionobjects.ValueFunction;

/**
 * Scalar operator for scalar-matrix operations with scalar 
 * on the right-hand-side.
 */
public class RightScalarOperator extends ScalarOperator 
{
	private static final long serialVersionUID = 5148300801904349919L;
	
	public RightScalarOperator(ValueFunction p, double cst) {
		super(p, cst);
	}

	@Override
	public void setConstant(double cst) 
	{
		super.setConstant(cst);
		
		//enable conditionally sparse safe operations
		sparseSafe |= (isSparseSafeStatic()
			|| (fn instanceof GreaterThan && _constant>=0)
			|| (fn instanceof GreaterThanEquals && _constant>0)
			|| (fn instanceof LessThan && _constant<=0)
			|| (fn instanceof LessThanEquals && _constant<0)
			|| (fn instanceof Divide && _constant!=0));
	}
	
	@Override
	public double executeScalar(double in) throws DMLRuntimeException {
		return fn.execute(in, _constant);
	}
	
	@Override
	protected boolean isSparseSafeStatic() {
		//add power as only rhs op sparse safe (1^0=1 but 0^1=0).
		return (super.isSparseSafeStatic() 
			|| fn instanceof Power);
	}
}

@@ -2041,16 +2041,60 @@
 ivide &&
+ _constant!=0)%0A%09%09%09%7C%7C (fn instanceof Power &&
  _consta
@@ -2102,16 +2102,16 @@
 t!=0));%0A
-
 %09%7D%0A%09%0A%09@O
@@ -2187,16 +2187,16 @@
 ption %7B%0A
+
 %09%09return
@@ -2230,190 +2230,6 @@
 %0A%09%7D%0A
-%09%0A%09@Override%0A%09protected boolean isSparseSafeStatic() %7B%0A%09%09//add power as only rhs op sparse safe (1%5E0=1 but 0%5E1=0).%0A%09%09return (super.isSparseSafeStatic() %0A%09%09%09%7C%7C fn instanceof Power);%0A%09%7D%0A
 %7D%0A

Fix correctness matrix-scalar pow (sparse-safeness)