Skip to content

Comments

add gc_count, total_base_count, and gc_percent to fastq-peek.sh#5

Open
stjacqrm wants to merge 1 commit intomainfrom
rmsj-ex1-dev
Open

add gc_count, total_base_count, and gc_percent to fastq-peek.sh#5
stjacqrm wants to merge 1 commit intomainfrom
rmsj-ex1-dev

Conversation

@stjacqrm
Copy link
Collaborator

Description

Fastq-peek.sh has been modified to include GC counts, total base counts, and GC percent as was requested in exercise01. GC percent prints to stdout as requested. READ_COUNT was also updated to more closely reflect the output from fastqc results.

Type of change

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also note any relevant details for your test configuration.

The code changes were tested using three samples; the sample provided in the sandbox, as well as two other openly available fastq files. All files were analyzed using FastQC to obtain base-level (all puns intended) quality scores. Fastq-peek.sh generated results that were concordant with FastQC.

Checklist:

  • My code adheres to the repository style guide
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings
  • New and existing tests pass locally with my changes

@stjacqrm stjacqrm requested a review from cimendes February 11, 2025 19:30
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.

1 participant