Completeness of Artificial Harmonics
cwitkowitz opened this issue · comments
Based on the current models for the artificial harmonics, I am wondering if TappedHarmonic
, PinchHarmonic
, and SemiHarmonic
should be inheriting from ArtificialHarmonic
instead of HarmonicEffect
? Otherwise, I think the data for these classes may be incomplete, but I could be wrong.
Hey @cwitkowitz! No, I don't think these three should inherit from ArtificialHarmonic. HarmonicEffect is an abstract class that is made specific by the concrete types of harmonics, including NaturalHarmonic. In particular, ArtificialHarmonic has information about the pitch and the octave of the harmonic, and this info is missing from TappedHarmonic, PinchHarmonic, and SemiHarmonic.
![](https://private-user-images.githubusercontent.com/1044300/273149343-29d179a6-1e90-421d-8494-255a746f5dad.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjI4MDc0NjIsIm5iZiI6MTcyMjgwNzE2MiwicGF0aCI6Ii8xMDQ0MzAwLzI3MzE0OTM0My0yOWQxNzlhNi0xZTkwLTQyMWQtODQ5NC0yNTVhNzQ2ZjVkYWQucG5nP1gtQW16LUFsZ29yaXRobT1BV1M0LUhNQUMtU0hBMjU2JlgtQW16LUNyZWRlbnRpYWw9QUtJQVZDT0RZTFNBNTNQUUs0WkElMkYyMDI0MDgwNCUyRnVzLWVhc3QtMSUyRnMzJTJGYXdzNF9yZXF1ZXN0JlgtQW16LURhdGU9MjAyNDA4MDRUMjEzMjQyWiZYLUFtei1FeHBpcmVzPTMwMCZYLUFtei1TaWduYXR1cmU9MDMzZTJiZDliMzVlMDU4YTcxODlkZjBmYWIyNmE0ODM2MTQzOGJiOTcxNDJjMzE3ZmMzODk3YWIwMjM4MjRhNyZYLUFtei1TaWduZWRIZWFkZXJzPWhvc3QmYWN0b3JfaWQ9MCZrZXlfaWQ9MCZyZXBvX2lkPTAifQ.wfA336YnyBc-_qy5zKOresgPVXaIIPPnLeUJaMCxD2U)