/*
 * Copyright (c) 2017, SRI International
 * All rights reserved.
 * Licensed under the The BSD 3-Clause License;
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at:
 * 
 * http://opensource.org/licenses/BSD-3-Clause
 * 
 * Redistribution and use in source and binary forms, with or without
 * modification, are permitted provided that the following conditions
 * are met:
 * 
 * Redistributions of source code must retain the above copyright
 * notice, this list of conditions and the following disclaimer.
 * 
 * Redistributions in binary form must reproduce the above copyright
 * notice, this list of conditions and the following disclaimer in the
 * documentation and/or other materials provided with the distribution.
 * 
 * Neither the name of the aic-expresso nor the names of its
 * contributors may be used to endorse or promote products derived from
 * this software without specific prior written permission.
 * 
 * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
 * "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
 * LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS
 * FOR A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE
 * COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, 
 * INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES 
 * (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
 * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) 
 * HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, 
 * STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) 
 * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED
 * OF THE POSSIBILITY OF SUCH DAMAGE.
 */
package com.sri.ai.grinder.sgdpllt.interpreter;

import java.util.Iterator;
import java.util.List;
import java.util.Map;
import java.util.Random;

import com.sri.ai.expresso.api.Expression;
import com.sri.ai.grinder.helper.AssignmentsSamplingIterator;
import com.sri.ai.grinder.sgdpllt.api.Context;
import com.sri.ai.grinder.sgdpllt.group.AssociativeCommutativeGroup;
import com.sri.ai.grinder.sgdpllt.library.controlflow.IfThenElse;
import com.sri.ai.grinder.sgdpllt.rewriter.api.Rewriter;
import com.sri.ai.grinder.sgdpllt.rewriter.api.TopRewriter;

/**
 * 
 * @author oreilly
 *
 */
public class SampleMultiIndexQuantifierEliminator extends AbstractIterativeMultiIndexQuantifierElimination {
	private int sampleSizeN;
	private Rewriter indicesConditionRewriter;
	private Random random;
	
	public SampleMultiIndexQuantifierEliminator(TopRewriter topRewriter, int sampleSizeN, Rewriter indicesConditionRewriter, Random random) {
		super(topRewriter);
		this.sampleSizeN = sampleSizeN;
		this.indicesConditionRewriter = indicesConditionRewriter;
		this.random = random;
	}
	
	public SampleMultiIndexQuantifierEliminator(TopRewriterUsingContextAssignments topRewriterWithBaseAssignment, int sampleSizeN, Rewriter indicesConditionRewriter, Random random) {
		super(topRewriterWithBaseAssignment);
		this.sampleSizeN = sampleSizeN;
		this.indicesConditionRewriter = indicesConditionRewriter;
		this.random = random;
	}
	
	@Override
	public Iterator<Map<Expression, Expression>> makeAssignmentsIterator(List<Expression> indices, Expression indicesCondition, Context context) {
		return new AssignmentsSamplingIterator(indices, sampleSizeN, indicesCondition, indicesConditionRewriter, random, context);
	}
	
	@Override
	public Expression makeSummand(AssociativeCommutativeGroup group, List<Expression> indices, Expression indicesCondition, Expression body, Context context) {
		return IfThenElse.make(indicesCondition, body, group.additiveIdentityElement());
	}
}
@@ -2153,74 +2153,8 @@
 up;%0A
-import com.sri.ai.grinder.sgdpllt.library.controlflow.IfThenElse;%0A
 impo
@@ -3592,87 +3592,138 @@
 %7B%0A%09%09
-return IfThenElse.make(indicesCondition, body, group.additiveIdentityElement())
+// NOTE: the AssignmentsSamplingIterator takes the indicesCondition into account so no need to take it into account.%0A%09%09return body
 ;%0A%09%7D

Correct makeSummand() logic.