Diaz335 / plntpth8300_final_project

0 stars 0 forks source link

Check out my project #5

Open Diaz335 opened 3 years ago

Diaz335 commented 3 years ago

@jelmerp Would you still be available Monday during office hours?

jelmerp commented 3 years ago

Hi Josue,

As mentioned in my email, I would happy to meet sometime soon.

Aspect Max. points Your points
Project organization 2 1 See below.
Project background and documentation 2 1.5
Script documentation 2 1.5
Good practices in scripts 4 2 See below.
Coding quality and complexity 3 2.5
Workflow reproducibility 3 2 See below.
Version control 3 2.5
SLURM jobs at OSC 2 2
TOTAL 20 15

Your Markdown documents are really useful to have as additional documentation; they are basically a digital lab notebook and it is good that you noted everything you did in there. However, you did not take the necessary steps to create separate executable scripts (actual .sh and .py files that can be directly run from the command-line) + a workflow script that can directly rerun the project.

Moreover, your Markdown documents, as rendered on GitHub, are very hard to read. For instance, the code is not put in code blocks, and line breaks are not properly displayed.