search

modelica-tools / csv-compare

Tool to compare curves from one csv files with curves from other csv files using an adjustable tolerance
https://www.modelica.org
BSD 3-Clause "New" or "Revised" License
25 stars 15 forks source link

Incorrect handling of result points at the same timepoint #4

Closed ghost closed 9 years ago

ghost commented 9 years ago

We previously created this ticket. It was fixed with the new algorithm. However we still see some odd behaviours, probably related to input handling. fig If you check the result points you see that the result points at t = 0.1 has been swapped.

Both the base and result has been swapped in the example above, however there are other places in that signal where only the result is swapped, t=~0.148, t=~0.1965, t=~2446, etc.

Command:

Compare.exe -o -m csvFileCompare -d , actual.csv expected.csv Files can be found here This was found using this version

svenruetz commented 9 years ago

In my development version (should be the same I uploaded) it looks like this: report 26 01 I used your files and your commandline

ghost commented 9 years ago

I've tested with both the 32bit and 64bit versions from here. I get the following version information if I run without arguments, notice that it says 2.0.0.1:

CSV File Comparison Tool 2.0.0.1 Copyright © 2015 - ITI GmbH

svenruetz commented 9 years ago

This is really weird, I even downloaded the zips and tried them and I cannot reproduce your plot. I even changed my system settings to swedish :)

Can you send me the report html files so I can check the arrays in the javascript part of the report.

ghost commented 9 years ago

Ok so I'm pretty sure that this is browser/jqplot related. I get this in IE 11.0.9600.17420: ie fig The image above was rendered using Chrome 39.0.2171.95 m I've uploaded the html file here

svenruetz commented 9 years ago

Jep. Browser issue, I can reproduce in current IE and Chrome. As it is "only" a plotting error I'll try to update the jqplot lib to a current version; hopefully this will fix the problem

svenruetz commented 9 years ago

I updated jqplot and I found the exact value where the plotting fails in Chrome and IE but the problem is still there.

May I use the report with your csv files to file a bug at jqplot?

ghost commented 9 years ago

Ok, sad to hear that it didn't resolve the issue. You may use the csv files for a jqplot issue, no problem.

svenruetz commented 9 years ago

I created a ticket for jqplot here: https://bitbucket.org/cleonello/jqplot/issue/1013/wrong-display-of-plots-across-browsers