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.

Sign up for GitHub

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

Jump to bottom

Minor fixes ahead of Tech Lab 2024 #37

Open

ansonmiu0214 wants to merge 4 commits into bloomberg:main from ansonmiu0214:fix

ansonmiu0214 commented Apr 22, 2024 •

edited

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.

ansonmiu0214 added 4 commits

April 22, 2024 17:34


          Fix images

01c5231


          Fix installation.md

6b7e0a2


          Ignore .ipynb_checkpoints

8dad586


          Update notebooks to always fix system path

04ca5f6

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment