promworkbench / BasicStochasticMiner

0 stars 0 forks source link

Unsure if a bug, but can lead to ConvergenceException in Optimizer. #2

Open AdamBanham opened 5 months ago

AdamBanham commented 5 months ago

In building our stochastic discovery, I often encountered a convergence exception.

I am unsure if this is an acceptable solution, but after some testing, I found if I changed the following line to reset the param to lowest or check that the parameter isn't lower than lowest, the convergence exception could be avoided.

https://github.com/promworkbench/BasicStochasticMiner/blob/8d1485cab435ba51753820149d6e6e01a7fc5cca/src/org/processmining/basicstochasticminer/solver/Solver.java#L89-L91

A change that worked for me was the one of the following:

} else if (params.getEntry(i) < lowest) {
    params.setEntry(i, 0);
}

or


} else if (params.getEntry(i) < lowest) {
     params.setEntry(i, lowest * 2);
}```
The multiplied number is somewhat arbitrary. 
AdamBanham commented 5 months ago

The second edit reduces the number of bads sent to Sysout to zero in my case.