Fix import of fastqc data - Githubissues

eastgenomics / trendyQC

Django app for monitoring trends in MultiQC data

MIT License

0 stars 0 forks source link

Fix import of fastqc data #67

Closed Yu-jinKim closed 7 months ago

Yu-jinKim commented 7 months ago

Fixes #66
Added fields in the Fastqc model to link all 4 lane/read combinations to samples
Separate setting up and importing of report objects

This change is

pep8speaks commented 7 months ago

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

In the file trendyqc/trend_monitoring/management/commands/utils/_multiqc.py:

Line 388:80: E501 line too long (81 > 79 characters) Line 392:80: E501 line too long (81 > 79 characters) Line 451:80: E501 line too long (82 > 79 characters) Line 483:80: E501 line too long (80 > 79 characters)

In the file trendyqc/trend_monitoring/models/fastq_qc.py:

Line 5:80: E501 line too long (118 > 79 characters) Line 6:80: E501 line too long (118 > 79 characters) Line 7:80: E501 line too long (118 > 79 characters) Line 8:80: E501 line too long (118 > 79 characters)