Skip to content

Commit

Permalink
Update nucleotide-count.jl comment (JuliaLang#329)
Browse files Browse the repository at this point in the history
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.
  • Loading branch information
tripitakit authored Feb 16, 2021
1 parent a63578d commit b99bcf9
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion exercises/practice/nucleotide-count/nucleotide-count.jl
Original file line number Diff line number Diff line change
@@ -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`.
Expand Down

0 comments on commit b99bcf9

Please sign in to comment.