jeroenraven / cspsol

Automatically exported from code.google.com/p/cspsol
GNU General Public License v3.0
0 stars 0 forks source link

XML pattern count may be incorrect #5

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
XML report may under report the pattern count
ie: pattern count may be 4 but only 3 pattern XML reps in the report.

file:
reports.cpp
void BBNode::print_xml_report(ostream& fout, glp_prob * master_lp, 
OrderWidthContainer& ow_set) 

x (the pattern count) is defined as a double.

change

for (patcnt=0;patcnt<x;patcnt++) {

to

for (patcnt=0;patcnt<lround(x);patcnt++) {

Original issue reported on code.google.com by jonathan...@yahoo.com on 27 Oct 2009 at 3:26

GoogleCodeExporter commented 8 years ago
thanks for reporting this. Will soon commit this change with other improvements.

Original comment by vijay.pa...@gmail.com on 29 Oct 2009 at 3:50

GoogleCodeExporter commented 8 years ago
This is still an issue. Not committed in latest version.

Original comment by jonathan...@yahoo.com on 25 Nov 2009 at 1:27

GoogleCodeExporter commented 8 years ago
Bug fix commited.
http://code.google.com/p/cspsol/source/detail?r=41

I was waiting for a chance to have detail look at this file and improve it 
along with
the bug fix.

Original comment by vijay.pa...@gmail.com on 25 Nov 2009 at 3:43