Mantis Bug Tracker

View Issue DetailsJump to Notes ] Wiki ] << ] >> ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0000344jquantlib[All Projects] code reviewpublic2009-08-20 11:452010-03-09 01:18
ReporterRichard Gomes 
Assigned ToRichard Gomes 
PriorityurgentSeverityminorReproducibilityhave not tried
StatusassignedResolutionopen 
PlatformOSOS Version
Product Version 
Target Version0.2.4Fixed in Version 
Summary0000344: Possible bug in Calendars, regarding isBusinessDay
DescriptionA user reported a possible bug. Please investigate:

In Calendar, I wonder if I have some misunderstandings about the API addHoliday. Once we call addHoliday(date) to add date to HolidayList, won't isBusinessDay(date) return false? Take ChinaSSECalendar as an example, I think the override method isBusinessDay should call the one in Super class to judge if the date is contained in addHolidayList?
TagsNo tags attached.
Attached Files? file icon JQLibTestSample.java [^] (1,446 bytes) 2009-08-20 11:45

- Relationships
parent of 0000356assignedRichard Gomes The implementation of isEndOfMonth method in AbstractCalendar class is not correct 
related to 0000327resolvedZahid Hussain Rewrite Date and Calendar classes from scratch 
Not all the children of this issue are yet resolved or closed.

-  Notes
(0000431)
Richard Gomes (manager)
2009-08-26 12:01

Analyzing QuantLib sources, it seems like trading calendars does not support addition/removal of dates.

Actually, QuantLib API allows you to call add/remove, which will insert/remove a certain date to the underlying holidayList as you expect but isBusinessDay() does not query this data structure. If you create your own Calendar, it will work as expected but pre-defined calendars override isBusinessDay() method.
(0000432)
Richard Gomes (manager)
2009-08-26 12:04

Gary,

If you have a chance, could you please confirm my diagnostic?

I only analyzed the problem and found an explanation for it.
I haven't created any small application to confirm how QuantLib behaves.

Thanks
(0000730)
Richard Gomes (manager)
2009-12-19 19:41

Reminder sent to: Gary Kennedy, Richard Gomes, Srinivas Hasti, Zahid Hussain

Waiting for feedback in order to be closed.
(0000767)
Richard Gomes (manager)
2010-03-09 01:17

No feedback from Gary :(

Kicked to v0.1.4

- Issue History
Date Modified Username Field Change
2009-08-20 11:45 Richard Gomes New Issue
2009-08-20 11:45 Richard Gomes Status new => assigned
2009-08-20 11:45 Richard Gomes Assigned To => Richard Gomes
2009-08-20 11:45 Richard Gomes File Added: JQLibTestSample.java
2009-08-20 12:16 Richard Gomes Relationship added related to 0000327
2009-08-26 12:01 Richard Gomes Note Added: 0000431
2009-08-26 12:04 Richard Gomes Note Added: 0000432
2009-08-26 12:04 Richard Gomes Assigned To Richard Gomes => Gary Kennedy
2009-08-26 12:04 Richard Gomes Status assigned => feedback
2009-11-01 01:47 Richard Gomes Status feedback => assigned
2009-11-01 01:47 Richard Gomes Assigned To Gary Kennedy => Richard Gomes
2009-12-19 19:40 Richard Gomes Relationship added parent of 0000356
2009-12-19 19:41 Richard Gomes Issue Monitored: Gary Kennedy
2009-12-19 19:41 Richard Gomes Issue Monitored: Srinivas Hasti
2009-12-19 19:41 Richard Gomes Issue Monitored: Zahid Hussain
2009-12-19 19:41 Richard Gomes Note Added: 0000730
2010-03-09 01:17 Richard Gomes Note Added: 0000767
2010-03-09 01:18 Richard Gomes Target Version 0.1.3 => 0.1.4


Copyright © 2000 - 2009 MantisBT Group
Powered by Mantis Bugtracker