Skip to content
This repository was archived by the owner on May 29, 2025. It is now read-only.

ANA-4464: instrument event recipe support#808

Open
nd368 wants to merge 1 commit intofinbourne:developfrom
nd368:feature/ANA-4464.v2
Open

ANA-4464: instrument event recipe support#808
nd368 wants to merge 1 commit intofinbourne:developfrom
nd368:feature/ANA-4464.v2

Conversation

@nd368
Copy link

@nd368 nd368 commented Oct 16, 2024

Pull Request Checklist

  • Changes follow the style guide
  • Tests pass
  • Raised the PR against the develop branch
  • Notebook outputs do not contain any priviledged data

Description of the PR

Modify MBS, IRS, FRA notebooks so that they correctly configure an instrument events recipe. This is required for an upcoming breaking change in which we stop falling back to the valuation recipe.

Tested against local LUSID - notebooks work before/after the breaking change, and with/without instrument events enabled.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@nd368 nd368 changed the base branch from master to develop October 16, 2024 14:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments