broadinstitute / pooled-cell-painting-profiling-template

:hammer_and_wrench: Use me to version control Pooled Cell Painting data and processing pipelines
BSD 3-Clause "New" or "Revised" License
2 stars 3 forks source link

Major configuration update #11

Closed gwaybio closed 4 years ago

gwaybio commented 4 years ago

This pull request represents a major update and breaking change to the configuration file processing. The PR addresses 2 of 3 (and lays the groundwork for the remaining 1/3) of #10.

The PR also simultaneously begins the foray into multi-plate options as described in broadinstitute/pooled-cell-painting-profiling-recipe#50.

Specific Updates

Next Steps

Once this pull request is digested and reviewed, we will need to perform several things:

  1. We need to update the config documentation files. I do not think much will need to be added, just rearranged.
  2. We will need to complete the multi-plate processing strategy - this PR is only half way multi-plate
gwaybio commented 4 years ago

the last two commits add the command argument parser utility function from the recipe to the template. The template repo will need the command args for the multi-plate processing updated weld procedure

ErinWeisbart commented 4 years ago

We need to update the config documentation files. I do not think much will need to be added, just rearranged.

I'm happy to deal with the documentation update.

gwaybio commented 4 years ago

thanks for the helpful comments @ErinWeisbart - I addressed them in the recent commits. PR ready for another look when you get a chance

ErinWeisbart commented 4 years ago