eastgenomics / trendyQC

Django app for monitoring trends in MultiQC data
MIT License
0 stars 0 forks source link

Move some functions from MultiQC object to script + use find longest common substring to merge data using sample name #65

Closed Yu-jinKim closed 5 months ago

Yu-jinKim commented 5 months ago

My previous function would only work when 2 sample names were overlapping but not more. The logic is to find the longest common substring and use it as the new sample name for the rest of the data


This change is Reviewable

pep8speaks commented 5 months ago

Hello @Yu-jinKim! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 10:1: E302 expected 2 blank lines, found 1

Comment last updated at 2024-02-02 11:48:40 UTC