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

FL-1323: FX Forward Notebook changes#804

Open
kr0bfb wants to merge 1 commit intofinbourne:masterfrom
kr0bfb:FL-1323
Open

FL-1323: FX Forward Notebook changes#804
kr0bfb wants to merge 1 commit intofinbourne:masterfrom
kr0bfb:FL-1323

Conversation

@kr0bfb
Copy link

@kr0bfb kr0bfb commented Oct 8, 2024

FL-1323:

- added numbering to the headinggs
- Extend the valuations dataset
- Added second “Valuation Output” section with a recipe which doesn’t split the legs

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

Describe the code changes for the reviewers, explain the solution you have provided and how it fixes the issue

FL-1323:

    - added numbering to the headinggs
    - Extend the valuations dataset
    - Added second “Valuation Output” section with a recipe which doesn’t split the legs
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

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