Skip to content

Commit 965c284

Browse files
committed
fixed bug where viz was still trace for static models
1 parent a85c1a8 commit 965c284

File tree

4 files changed

+4
-4
lines changed

4 files changed

+4
-4
lines changed

org.alloytools.alloy.application/src/main/java/edu/mit/csail/sdg/alloy4whole/SimpleCLI.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ public void bound(String msg) {
118118

119119
@Override
120120
public void translate(String solver, int bitwidth, int maxseq, int mintrace, int maxtrace, int skolemDepth, int symmetry, String strat) {
121-
debug("Solver=" + solver + " Steps=" + mintrace + ".." + maxtrace + " Bitwidth=" + bitwidth + " MaxSeq=" + maxseq + " Symmetry=" + (symmetry > 0 ? ("" + symmetry) : "OFF") + " Mode=" + strat + "\n");
121+
debug("Solver=" + solver + (maxtrace < 1 ? "" : " Steps=" + mintrace + ".." + maxtrace) + " Bitwidth=" + bitwidth + " MaxSeq=" + maxseq + " Symmetry=" + (symmetry > 0 ? ("" + symmetry) : "OFF") + " Mode=" + strat + "\n");
122122
}
123123

124124
@Override

org.alloytools.alloy.application/src/main/java/edu/mit/csail/sdg/alloy4whole/SimpleMain.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -93,7 +93,7 @@ public void bound(String msg) {
9393

9494
@Override
9595
public void translate(String solver, int bitwidth, int maxseq, int mintrace, int maxtrace, int skolemDepth, int symmetry, String strat) {
96-
info("Solver=" + solver + " Steps=" + mintrace + ".." + maxtrace + " Bitwidth=" + bitwidth + " MaxSeq=" + maxseq + " Symmetry=" + (symmetry > 0 ? ("" + symmetry) : "OFF") + " Mode=" + strat + "\n");
96+
info("Solver=" + solver + (maxtrace < 1 ? "" : " Steps=" + mintrace + ".." + maxtrace) + " Bitwidth=" + bitwidth + " MaxSeq=" + maxseq + " Symmetry=" + (symmetry > 0 ? ("" + symmetry) : "OFF") + " Mode=" + strat + "\n");
9797
}
9898

9999
int totalVars = 0, totalPvars = 0, totalClauses = 0, lastStep = 0;

org.alloytools.alloy.application/src/main/java/edu/mit/csail/sdg/alloy4whole/SimpleReporter.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -382,7 +382,7 @@ public void debug(final String msg) {
382382
public void translate(String solver, int bitwidth, int maxseq, int mintrace, int maxtrace, int skolemDepth, int symmetry, String strat) {
383383
startTime = System.currentTimeMillis();
384384
startCount = 0;
385-
cb("translate", "Solver=" + solver + " Steps=" + mintrace + ".." + maxtrace + " Bitwidth=" + bitwidth + " MaxSeq=" + maxseq + (skolemDepth == 0 ? "" : " SkolemDepth=" + skolemDepth) + " Symmetry=" + (symmetry > 0 ? ("" + symmetry) : "OFF") + " Mode=" + strat + "\n");
385+
cb("translate", "Solver=" + solver + (maxtrace < 1 ? "" : " Steps=" + mintrace + ".." + maxtrace) + " Bitwidth=" + bitwidth + " MaxSeq=" + maxseq + (skolemDepth == 0 ? "" : " SkolemDepth=" + skolemDepth) + " Symmetry=" + (symmetry > 0 ? ("" + symmetry) : "OFF") + " Mode=" + strat + "\n");
386386
}
387387

388388
/** {@inheritDoc} */

org.alloytools.alloy.core/src/main/java/edu/mit/csail/sdg/translator/A4SolutionReader.java

+1-1
Original file line numberDiff line numberDiff line change
@@ -432,7 +432,7 @@ private A4SolutionReader(Iterable<Sig> sigs, XMLNode xml) throws IOException, Er
432432
A4Options opt = new A4Options();
433433
opt.originalFilename = inst.getAttribute("filename");
434434
// [electrum] do not use actual max trace, solution would identify unbounded solving but no unbounded solver
435-
sol = new A4Solution(inst.getAttribute("command"), bitwidth, mintrace, tracelength, maxseq, strings, atoms, null, opt, 1);
435+
sol = new A4Solution(inst.getAttribute("command"), bitwidth, Math.min(tracelength, mintrace), Math.min(tracelength, maxtrace), maxseq, strings, atoms, null, opt, 1);
436436
factory = sol.getFactory();
437437
// parse all the sigs, fields, and skolems
438438
for (Map.Entry<String,XMLNode> e : nmap.entrySet())

0 commit comments

Comments
 (0)