jmaciuch / Cov2_Analysis

0 stars 0 forks source link

Analysis Plan Feedback #4

Closed kescobo closed 3 years ago

kescobo commented 3 years ago

:exclamation: Don't merge this! :exclamation:

This PR is for feeback related to your analysis plan

Overall

Plan 1

Plan 2

Rubrics

Excellent - full credit (5 pts)

Good - partial credit (4 pts)

Needs work - partial credit (3 or fewer points)

kescobo commented 3 years ago

Looks great! I left a few comments / suggestion, but overall this looks like a good and doable set of analyses. The only thing missing is that you didn't describe how you're planning to display the results from analysis 2.

jmaciuch commented 3 years ago

Looks great! I left a few comments / suggestion, but overall this looks like a good and doable set of analyses. The only thing missing is that you didn't describe how you're planning to display the results from analysis 2.

Looks great! I left a few comments / suggestion, but overall this looks like a good and doable set of analyses. The only thing missing is that you didn't describe how you're planning to display the results from analysis 2.

I mentioned it in step 4: """ Create function that plots alignment score vs. length of time between collection dates.

Sorry if that wasn't clear. The plan is to make a function that uses the functions in step 2 & 3 to make tuples containing (length of time between collections, alignment score), and then use those as coordinate points on a scatter plot with time on x-axis and alignment score on y

kescobo commented 3 years ago

Quite right! Skimmed right over that it seems...

You might also consider boxplots (really, something like a candlestick plot) depending on the number of sequences you end up looking at and their dispersion, but starting with a scatter definitely makes sense :+1: