lib/head_music/scale.rb in head_music-0.5.3 vs lib/head_music/scale.rb in head_music-0.5.4

- old
+ new

@@ -17,11 +17,11 @@ def pitches(direction: :ascending, octaves: 1) @pitches ||= {} @pitches[direction] ||= {} @pitches[direction][octaves] ||= begin - letter_cycle = root_pitch.letter_cycle + letter_name_cycle = root_pitch.letter_name_cycle semitones_from_root = 0 [root_pitch].tap do |pitches| if [:ascending, :both].include?(direction) (1..octaves).each do |i| scale_type.ascending_intervals.each_with_index do |semitones, i| @@ -48,12 +48,12 @@ def pitch_names(direction: :ascending, octaves: 1) pitches(direction: direction, octaves: octaves).map(&:name) end - def letter_cycle - @letter_cycle ||= root_pitch.letter_cycle + def letter_name_cycle + @letter_name_cycle ||= root_pitch.letter_name_cycle end def root_pitch_number @root_pitch_number ||= root_pitch.number end @@ -75,21 +75,21 @@ end def letter_for_step(step, semitones_from_root, direction) pitch_class_number = (root_pitch.pitch_class.to_i + semitones_from_root) % 12 if scale_type.intervals.length == 7 - direction == :ascending ? letter_cycle[step % 7] : letter_cycle[-step % 7] + direction == :ascending ? letter_name_cycle[step % 7] : letter_name_cycle[-step % 7] elsif scale_type.intervals.length < 7 && parent_scale_pitches - parent_scale_pitch_for(semitones_from_root).letter + parent_scale_pitch_for(semitones_from_root).letter_name elsif root_pitch.flat? HeadMusic::PitchClass::FLAT_SPELLINGS[pitch_class_number] else HeadMusic::PitchClass::SHARP_SPELLINGS[pitch_class_number] end end def pitch_for_step(step, semitones_from_root, direction) pitch_number = root_pitch_number + semitones_from_root - letter = letter_for_step(step, semitones_from_root, direction) - HeadMusic::Pitch.from_number_and_letter(pitch_number, letter) + letter_name = letter_for_step(step, semitones_from_root, direction) + HeadMusic::Pitch.from_number_and_letter(pitch_number, letter_name) end end