Patches item #2076218, was opened at 2008-08-26 18:16
Message generated for change (Comment added) made by lballabio
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=312740&aid=2076218&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: Accepted
Priority: 5
Private: No
Submitted By: Slava Mazur (shlagbaum)
>Assigned to: Luigi Ballabio (lballabio)
Summary: Period::frequency() changed
Initial Comment:
Handling of week and day time units has changed in Period::frequency(). No exception is thrown in response to incorrect input. NoFrequency is returned instead. The acceptable range of lengths is widen for week and day time units: weeks - 1,2,4,13,26,52; days - 1,7,14,91,182,365.
----------------------------------------------------------------------
>Comment By: Luigi Ballabio (lballabio)
Date: 2008-09-02 11:55
Message:
Logged In: YES
user_id=75450
Originator: NO
The modifications:
1) Instead of using NoFrequency, a further enum (OtherFrequency) was
added. This specifies some unknown but regular frequency, while NoFrequency
stands for no frequency at all.
2) The known range of lengths was not widened. I preferred to keep the
invariant that Period(Frequency(p)) either fails or returns the original p.
This would not have been the case if the acceptable lengths were increased;
for instance, Period(Frequency(26*Weeks)) would have returned 6*Months
which doesn't equal 26*Weeks.
----------------------------------------------------------------------
Comment By: Luigi Ballabio (lballabio)
Date: 2008-09-02 11:55
Message:
Logged In: YES
user_id=75450
Originator: NO
The patch was applied (with some modifications) to the code repository.
It will be included in next release.
Thank you.
----------------------------------------------------------------------
You can respond by visiting:
https://sourceforge.net/tracker/?func=detail&atid=312740&aid=2076218&group_id=12740-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/_______________________________________________
QuantLib-dev mailing list
[hidden email]
https://lists.sourceforge.net/lists/listinfo/quantlib-dev
Locked
