Opened 17 years ago
Closed 17 years ago
#560 closed defect (remind)
Error whit SubSer function
Reported by: | lmperez | Owned by: | Víctor de Buen Remiro |
---|---|---|---|
Priority: | highest | Milestone: | |
Component: | TimeSetAlgebra | Version: | 1.1.6 |
Severity: | blocker | Keywords: | |
Cc: |
Description
There is a mistake with SubSer function, look at this:
Serie a = SubSer(CalInd(C,C),y2000m01d01,y2001m01d01);
Date b = y2001m02d03;
Date c = y2001m02d19;
Serie d = SubSer(a,b,c);
If you make a SubSer of a serie between two dates that are not defined in the timeset of the serie, the SubSer function creates a new serie defined between the las date of the serie, and the first date of the SubSer dates.
I think it's better if SubSer function stop whit an error notice instead of an unreal serie.
Change History (2)
comment:1 Changed 17 years ago by
Owner: | changed from danirus to Víctor de Buen Remiro |
---|
comment:2 Changed 17 years ago by
Resolution: | → remind |
---|---|
Status: | new → closed |
This is not an unreal time serie. A time serie is an infinite succession of data related to an infinite succession of time periods. Some times, you only can observe a bounded data window and the rest of values are unknown for you. Any serie with end date before begin date is the empty time serie. A time serie which data are all unknown is also the empty time serie. The empty time serie exists and is usefull to extend the concatenation related operators and other ones.
I think that it may be very confusing for in TOL beginners that the dates of the empty time series being any pair of reverse ordered dates. Probably, it could be more explicit if TOL uses allways the empty interval (TheEnd,TheBegin). I don't see any other problem with this feature and I can ensure you that this is no an error to be notified.
However this change could cause backward compatibility problems and it should be pass a very hard test in a large set of projects. If you want we can open a thread about this theme in tol user's mail lists to analyze this posibility.
Thanks for reporting.