xjiang4 / ellipsoids

Automatically exported from code.google.com/p/ellipsoids
Other
0 stars 1 forks source link

ReachContinuous.evolve creates tubes with duplicated time entries #112

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
evolve uses cat which glues two different tubes on time spans [a,b] and [b,c] 
(see end of evolve methods in ReachContinuous). This results into the 
duplicated b value in the resulting time grid which is incorrect.

Original issue reported on code.google.com by heartofm...@gmail.com on 24 Apr 2013 at 4:12

GoogleCodeExporter commented 8 years ago

Original comment by kirill.m...@gmail.com on 24 May 2013 at 3:06

GoogleCodeExporter commented 8 years ago
1) in ReachContinuous

indVec on line 931 is logical vector, where is "is" prefix?

2) Please add the following check into

gras.ellapx.smartdb.rels.EllTubeBasic.checkTuple:

[~,indSortVec]=unique(timeVec);%unique sorts input values
if length(indSortVec)~=length(timeVec);
    errTagStr='timeVecNotUnq';
    reasonStr='timeVec contains non-unique values';
    return;
elseif any(diff(indSortVec)<0)
    errTagStr='timeVecNotMon';
    reasonStr='timeVec is not monotone';
    return;
end

3) We need to merge with Igor's changes first (after he reintegrates his 
branch).

Original comment by heartofm...@gmail.com on 24 May 2013 at 3:58

GoogleCodeExporter commented 8 years ago

Original comment by kirill.m...@gmail.com on 26 May 2013 at 3:40

GoogleCodeExporter commented 8 years ago

Original comment by heartofm...@gmail.com on 27 May 2013 at 1:22

GoogleCodeExporter commented 8 years ago

Original comment by kirill.m...@gmail.com on 27 May 2013 at 5:08

GoogleCodeExporter commented 8 years ago
I'll close this once issue 108 is fixed.

Original comment by heartofm...@gmail.com on 28 May 2013 at 8:59

GoogleCodeExporter commented 8 years ago
Reintegrated via issue_108

Original comment by kirill.m...@gmail.com on 29 May 2013 at 12:29

GoogleCodeExporter commented 8 years ago

Original comment by heartofm...@gmail.com on 30 May 2013 at 1:15