Ophionodus / qbb2023-answers

0 stars 0 forks source link

Week 6 Feedback #4

Open dtaylo95 opened 10 months ago

dtaylo95 commented 10 months ago

README.md with commands and analyses

2/2

Exercise Points Possible Grade
Commands for Step 1.1 0.33 0.33
Commands for Step 2.1 0.33 0.33
Commands for Step 3.1 0.33 0.33
Answer to Step 3.4 1 1

plotting.py script to produce plots

3.25/4

Exercise Points Possible Grade
Code to produce step 1.2 PC plot 1 0.5
Code to produce step 2.2 AFS plot 1 1
Code to produce step 3.2 Manhattan plots 1 0.75
Code to produce step 3.3 effect size boxplot 1 1

Minor issue #1: When you're reading in the eigenvectors for the PCA plot, you never convert the values to floats (they are strings by default), so you're plotting strings, which is why your PCA plot looks so weird. For some reason, matplotlib doesn't through an error when you try to plot strings, it just does its best, which is wrong.

Minor issue #2: In your manhattan plot, you're plotting all of the significant points on the left of the plot, rather than in the appropriate spots relative to the other variants. When you extract the significant variants, you also want to extract their x-values, which can just be their index in the full list of variants.

Pretty plots

4/4

Exercise Points Possible Grade
Step 1.2 PC plot 1 1
Step 2.2 AFS plot 1 1
Step 3.2 Manhattan plots 1 1
Step 3.3 effect size boxplot 1 1

Grade

Total: 9.25/10

Great work! Just fix the minor issues and resubmit for the full 10/10

dtaylo95 commented 9 months ago

Regraded! Feel free to fix the minor issues and resubmit for the full 10/10