Open lilianabs opened 1 year ago
@lilianabs looks neat, nice work! I will test it out and share my comments here.
Could you pls post a screen shot of the docker container running and giving you a prompt as per the code - does not have to be exact or working, just want to see what it looks like on your end.
It's easy to take a screenshot and drag it into the comment box and it turns it into a image url to link to
Could you pls post a screen shot of the docker container running and giving you a prompt as per the code - does not have to be exact or working, just want to see what it looks like on your end.
It's easy to take a screenshot and drag it into the comment box and it turns it into a image url to link to
Thank you for reviewing the PR Mani! Here is a screenshot:
To be able to merge a pull request, there are a few checks:
DO NOT DELETE THE BELOW TEXT FROM THE BODY OF THE PULL REQUEST. ADD YOUR DESCRIPTION(S) FROM THE POINT IT SAYS "Goal or purpose of the PR". USE THIS TEMPLATE AS IS.
USE THE CHECKLIST TO SHOW PROGRESS AND A REMINDER FOR YOURSELF.
Checklist
Please check the options that you have completed and strike-out the options that do not apply via this pull request:
./test-coverage.sh "tests slow-tests"
) - this will also be visible via the Code coverage report and CI/CD task on the Pull Requestnotebooks
folder, read the Notebooks docs)Goal or purpose of the PR
This PR adds a docker image for the package development.