Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed read-bioinfo-metadata module #367

Merged
merged 31 commits into from
Jan 15, 2025

Conversation

svarona
Copy link
Member

@svarona svarona commented Jan 13, 2025

  • Made long table to be written in --outdir. Fix issue Longtable.json should be created in output-directory #351
  • Fixed module to find files not containing sample_name in filename (viralrecon_filepath_mapping_stats, viralrecon_filepath_summary_mqc, viralrecon_filepath_variants_long_table, viralrecon_filepath_workflow_summary)
  • Added year to --outdir to write files
  • Added batchID/date to filename instead of current date for --oudir and --batchdir
  • Added current date for tables written in analysis_results
  • Test whether variants_long_table and bioinfo_lab_metadata already exist and try to merge new samples, ignores samples already existing that contain exactly the same info.
  • Write long table only with samples in the COD to COD folder

@svarona svarona force-pushed the fix_bioinfo_metadata branch from 99a792b to 232071d Compare January 13, 2025 12:39
@svarona svarona marked this pull request as ready for review January 13, 2025 12:39
@svarona svarona force-pushed the fix_bioinfo_metadata branch from 232071d to 8ac625a Compare January 14, 2025 14:33
@svarona svarona force-pushed the fix_bioinfo_metadata branch from 00f192c to a480315 Compare January 14, 2025 15:44
relecov_tools/read_bioinfo_metadata.py Outdated Show resolved Hide resolved
@svarona svarona merged commit 030fd3a into BU-ISCIII:develop Jan 15, 2025
14 checks passed
@OPSergio OPSergio mentioned this pull request Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants