Posted by
SourceForge.net on
Nov 23, 2007; 8:40am
URL: http://quantlib.414.s1.nabble.com/quantlib-Bugs-1608340-Chinese-Calendars-bug-tp11423.html
Bugs item #1608340, was opened at 2006-12-04 10:02
Message generated for change (Comment added) made by lballabio
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=112740&aid=1608340&group_id=12740Please note that this message will contain a full copy of the comment thread,
including the initial issue submission, for this request,
not just the latest update.
Category: None
Group: None
>Status: Closed
>Resolution: Out of Date
Priority: 5
Private: No
Submitted By: Nobody/Anonymous (nobody)
>Assigned to: Luigi Ballabio (lballabio)
Summary: Chinese Calendars bug
Initial Comment:
The y argument in SpringFestival() substract 1900 twice. Thus the function references an illegal index of the array.
QuantLib-0.3.14\ql\Calendars:
bool China::Impl::isBusinessDay(const Date& date) const {
Weekday w = date.weekday();
Day d = date.dayOfMonth();
Month m = date.month();
Year y = date.year();
Day dd = date.dayOfYear();
if (isWeekend(w)
// New Year's Day
|| (d == 1 && m == January)
// Labor Day
|| (d >= 1 && d <= 7 && m == May)
// National Day
|| (d >= 1 && d <= 7 && m == October)
// Lunar New Year 2004
|| (d >= 22 && d <= 28 && m == January && y==2004)
// Spring Festival
|| (dd == springFestival(y-1900))
// ^^^^^ once
// Last day of Lunar Year
|| (dd == springFestival(y-1900)-1)
// ^^^^^ once
)
return false;
return true;
}
Day China::Impl::springFestival(Year y) {
static const Day SpringFestival[] = {
31, 51, 39, 29, 47, 36, 25, 44, 33, 22, // 1900-1909
41, 30, 49, 37, 26, 45, 34, 23, 42, 32, // 1910-1919
52, 39, 28, 47, 37, 24, 44, 33, 23, 41, // 1920-1929
30, 48, 37, 26, 45, 35, 24, 42, 31, 51, // 1930-1939
39, 27, 46, 37, 25, 44, 33, 22, 41, 29, // 1940-1949
48, 37, 27, 45, 34, 24, 43, 31, 49, 39, // 1950-1959
28, 46, 36, 25, 44, 33, 21, 40, 30, 48, // 1960-1969
37, 27, 46, 34, 23, 42, 31, 49, 38, 28, // 1970-1979
47, 36, 25, 44, 33, 51, 40, 29, 48, 37, // 1980-1989
27, 46, 36, 23, 41, 31, 51, 38, 28, 47, // 1990-1999
36, 24, 43, 32, 22, 40, 29, 49, 38, 26, // 2000-2009
45, 34, 23, 41, 31, 51, 39, 28, 47, 36, // 2010-2019
25, 43, 32, 22, 41, 29, 48, 37, 26, 44, // 2020-2029
34, 23, 42, 31, 50, 39, 28, 46, 36, 24, // 2030-2039
43, 32, 22, 41, 30, 48, 37, 26, 45, 33, // 2040-2049
23, 42, 32, 50, 39, 28, 46, 35, 24, 43, // 2050-2059
33, 21, 40, 29, 48, 36, 26, 45, 34, 23, // 2060-2069
42, 31, 51, 38, 27, 46, 36, 24, 43, 33, // 2070-2079
22, 40, 29, 48, 37, 26, 45, 35, 24, 41, // 2080-2089
30, 50, 38, 27, 46, 36, 25, 43, 32, 21, // 2090-2099
};
return SpringFestival[y-1900];
// ^^^^^ twice
}
----------------------------------------------------------------------
>Comment By: Luigi Ballabio (lballabio)
Date: 2007-11-23 09:40
Message:
Logged In: YES
user_id=75450
Originator: NO
Thanks for the bug report. However, recent code changes made it obsolete.
Luigi
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=112740&aid=1608340&group_id=12740-------------------------------------------------------------------------
This SF.net email is sponsored by: Microsoft
Defy all challenges. Microsoft(R) Visual Studio 2005.
http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/_______________________________________________
QuantLib-dev mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/quantlib-dev