Vincent Lostanlen
Vincent Lostanlen
@rabitt what are your thoughts?
Hello, what should we do exactly?
1. it's not obvious that we should do this. PyPI releases are application-oriented and typically do not include unit tests at all. If you need to run your own unit...
Agreed with 1. We need green light from @rabitt for 2.
Sounds good, many thanks. I don't understand the logic of `elif greek_index`, could you please explain?
Yes. See: https://github.com/rabitt/pysox/blob/master/sox/transform.py#L2769
i'm not sure if this is featured in pysox. it might be good to bring this up to @rabitt
Thanks. Could you write a unit test for this? How does this affect the behavior of pysox? Is it backwards compatible?
Thanks @dmitriilazukov. LGTM
@rabitt this is ready for merging IMO