007c fix - manifest tweaks

[KatyTaylor]

Closes #

Changes proposed in this pull request

• Change number of samples per pool to allow for larger pools, based on predicted samples coming through
• Add extra columns to manifest - info that is needed for data release
• (Feedback from Abby & Liz H in Slack)

Instructions for Reviewers

[All PRs] - Confirm PR template filled
[Feature Branches] - Review code
[Production Merges to main]
    - Check story numbers included
    - Check for debug code
    - Check version

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.13%. Comparing base (2529221) to head (4ed56ca).
Report is 8 commits behind head on develop.

Additional details and impacted files

@@             Coverage Diff             @@
##           develop    #4532      +/-   ##
===========================================
+ Coverage    89.10%   89.13%   +0.03%     
===========================================
  Files         1392     1392              
  Lines        29830    29830              
===========================================
+ Hits         26581    26590       +9     
+ Misses        3249     3240       -9     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[yoldas]

Notes:

I think the row_per_well value 120 arbitrary, just to get a higher number of samples, but it looks like a magic number.

The spreadsheet will be very long.

date_of_sample_collection and sample_ebi_accession_number is mapped to headings in spec/data/sample_manifest_excel/columns.yml as DATE OF SAMPLE COLLECTION and SAMPLE ACCESSION NUMBER (optional) respectively.