Skip to content
This repository was archived by the owner on Jul 1, 2026. It is now read-only.

Minor fixes ahead of Tech Lab 2024#37

Open
ansonmiu0214 wants to merge 4 commits into
bloomberg:mainfrom
ansonmiu0214:fix
Open

Minor fixes ahead of Tech Lab 2024#37
ansonmiu0214 wants to merge 4 commits into
bloomberg:mainfrom
ansonmiu0214:fix

Conversation

@ansonmiu0214

@ansonmiu0214 ansonmiu0214 commented Apr 22, 2024

Copy link
Copy Markdown

Issue number of the reported bug or feature request: #

Describe your changes

  • Installation.md:
    • Fix images
    • Add tip for finding full path of current directory, minor formatting fixes
  • .gitignore: Don't check in notebook checkpoints to Git
  • All portfolio manager notebooks: Put the path fix in the cells that set up ipytest, so the users don't have to do it

Testing performed

  • Images render fine now
  • Manually tested the Docker commands
  • Manually ran the cells in the notebook to confirm that the user's implementations can be resolved by the test driver

Additional context
Add any other context about your contribution here.

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