From b99bcf9aa355d0e665e27ace306fa813efa1303d Mon Sep 17 00:00:00 2001 From: Patrick De Marta Date: Tue, 16 Feb 2021 14:11:07 +0100 Subject: [PATCH] Update nucleotide-count.jl comment (#329) I got tricked by my own excitement beginning the julia track. I started solving the Nucleotide Count exercise without a complete reading of its introduction; the comment at the beginning seemed just enough: > count_nucleotides(strand) > The frequency of each nucleotide within `strand` as a dictionary. > Invalid strands raise a `DomainError`. So I read **frequency** and I think: count(nt) / length(strand) It was fun solving that.. Untill the tests revealed my mistake: we are asked to return the count of each nucleotide: not their frequencies. To avoid the confusion -for future students- I'd propose to replace "frequency" with "count" in the comment of the exercise's .jl file. --- exercises/practice/nucleotide-count/nucleotide-count.jl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/exercises/practice/nucleotide-count/nucleotide-count.jl b/exercises/practice/nucleotide-count/nucleotide-count.jl index cd5bb9acb742f..a75f3ffc7b5e4 100644 --- a/exercises/practice/nucleotide-count/nucleotide-count.jl +++ b/exercises/practice/nucleotide-count/nucleotide-count.jl @@ -1,7 +1,7 @@ """ count_nucleotides(strand) -The frequency of each nucleotide within `strand` as a dictionary. +The count of each nucleotide within `strand` as a dictionary. Invalid strands raise a `DomainError`.