Description
Excerpted from discussion in #1366:
@petertseng
The thing that gives me pause currently is that the propertychromatic
is about test cases with chromatic scales.
@coriolinus
I interpret the presence of the chromatic tests as a test of student understanding of music theory: chromatic tests are equivalent to interval tests with an interval of"mmmmmmmmmmmm"
.As a general policy I'm not a fan of exercises requiring external domain knowledge of students. At the same time, I recently implemented this exercise for the Rust track and didn't even notice that it required external domain knowledge.
I suggest that we pick from among the following options:
- decide that the knowledge of what constitutes a chromatic scale is general knowledge we can expect of the population, and therefore no action is required
- decide that the knowledge of what constitutes a chromatic scale is easily googleable, and therefore no action is required
- decide that knowledge of chromatic scales is domain knowledge, and therefore that the chromatic-property tests should be removed
@petertseng
I'd also consider the possibility of:Explicitly state in the README the need to be able to produce a chromatic scale.
I do think that the README already provides an explanation of what a chromatic scale is, though there's always the possibility that it could be clearer.