Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Changed paymentLag type to Integer #1818

Merged
merged 1 commit into from
Oct 26, 2023
Merged

Conversation

Stoozy
Copy link
Contributor

@Stoozy Stoozy commented Oct 24, 2023

No description provided.

@boring-cyborg
Copy link

boring-cyborg bot commented Oct 24, 2023

Thanks for opening this pull request! It might take a while before we look at it, so don't worry if there seems to be no feedback. We'll get to it.

@CLAassistant
Copy link

CLAassistant commented Oct 24, 2023

CLA assistant check
All committers have signed the CLA.

@Stoozy Stoozy mentioned this pull request Oct 24, 2023
@lballabio lballabio linked an issue Oct 24, 2023 that may be closed by this pull request
@coveralls
Copy link

coveralls commented Oct 24, 2023

Coverage Status

coverage: 72.016%. remained the same when pulling 90028ed on Stoozy:master into 6441d1b on lballabio:master.

@lballabio
Copy link
Owner

Thanks! A quick grep gives me a few more places that need to be modified:

ql/cashflows/fixedratecoupon.cpp:159:    FixedRateLeg& FixedRateLeg::withPaymentLag(Natural lag) {
ql/cashflows/fixedratecoupon.hpp:109:        FixedRateLeg& withPaymentLag(Natural lag);
ql/cashflows/fixedratecoupon.hpp:122:        Natural paymentLag_ = 0;
ql/cashflows/iborcoupon.cpp:178:    IborLeg& IborLeg::withPaymentLag(Natural lag) {
ql/cashflows/iborcoupon.hpp:140:        IborLeg& withPaymentLag(Natural lag);
ql/cashflows/iborcoupon.hpp:168:        Natural paymentLag_ = 0;
ql/cashflows/overnightindexedcoupon.cpp:303:    OvernightLeg& OvernightLeg::withPaymentLag(Natural lag) {
ql/cashflows/overnightindexedcoupon.hpp:104:        OvernightLeg& withPaymentLag(Natural lag);
ql/cashflows/overnightindexedcoupon.hpp:119:        Natural paymentLag_ = 0;
ql/cashflows/subperiodcoupon.cpp:188:    SubPeriodsLeg& SubPeriodsLeg::withPaymentLag(Natural lag) {
ql/cashflows/subperiodcoupon.hpp:129:        SubPeriodsLeg& withPaymentLag(Natural lag);
ql/cashflows/subperiodcoupon.hpp:152:        Natural paymentLag_ = 0;
ql/instruments/makeois.cpp:179:    MakeOIS& MakeOIS::withPaymentLag(Natural lag) {
ql/instruments/makeois.hpp:60:        MakeOIS& withPaymentLag(Natural lag);
ql/instruments/makeois.hpp:91:        Natural paymentLag_ = 0;

@lballabio
Copy link
Owner

@pcaspers: you mentioned swaps in the original issue, not bonds, right? Otherwise we'd also have to think about how accrual calculation should work in these cases.

@lballabio lballabio added this to the Release 1.33 milestone Oct 25, 2023
@pcaspers
Copy link
Contributor

Yes, cross currency swaps. I have not seen a bond with negative pay lag so far.

@lballabio lballabio merged commit 428bdd4 into lballabio:master Oct 26, 2023
51 checks passed
@boring-cyborg
Copy link

boring-cyborg bot commented Oct 26, 2023

Congratulations on your first merged pull request!

@lballabio
Copy link
Owner

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Negative Payment Lag
5 participants