Meeting Minutes

[zgarciaj]

Date: November 22, 2023

Attendees:

• Jerry Yu @jy1909
• John Shiu @JohnShiuMK
• Sophia Zhao @zth96
• Zeily Garcia @zgarciaj

Discussion Points:

• environment.yml
• function naming
• test data & comments in test.py
• folder structure (review tiff's github)
• report review
• README review
• branch merging
• Before leaving this

Decisions Made During Meeting:

• Environment.yml: John will go though the whole environment
• Everyone must run this and check it works
• Function naming: Action first eg. plot_correlation_heatmap
• test data & comments in test.py: Comment should be name of the function and then what we are testing the function on

Action Items:

• Review and Accept merge requests
• README and report updates
• Docker Hub credentials
• Potential Office Hours
• Questions:
  1. Some of our code is heavily manual (like model selection), do we have to wrap it in a function?
  2. Do our functions comply with “writing thorough function specifications & documentation”?
  3. Is it ok to create files in the test folder for testing data splitting function?
  4. Do we need to include the test folder in the docker image?
  5. Do we need to push the docker image, all of us? Or just one? (credentials)
  6. Is it ok if different tests use different approaches to raise the error?

Archive of our previous meeting minutes: https://docs.google.com/document/d/1z5eSyXdsCRY6X7Eg4lQtzfEE2GwEEi1fyIXwSWbUdig/edit?usp=sharing

Thanks @JohnShiuMK for the excellent recommendation of transitioning our meeting minutes to GitHub issues. This change significantly enhances their accessibility.