torstenanders Posted December 26, 2014 Share Posted December 26, 2014 The subject says it all... Best, Torsten LENGTH-DIVIDE2 Documentation seemingly not updated (neither reference nor examples) after function has been changed for the argument :position. For example, the following example call throws an error (length-divide2 '(1 2 2) '(1/16) '(1/4 1/4 1/4 1/4) :type 'n :position 'e) > Error: There is no POSITION named: e. > Allowed POSITION: 'f (first) 'l (last) 'c (center) and '? (at random). BTW: Unfortunately, that helpful error message is not shown if I try to directly output the above expression (e.g., with cmd-2). Instead it then says > Error: OMN Parse Error: (annotate (parse-omn-audition-phrase) (fail nil nil)) Ideally, when creating output the original exception with the much more clear error message would be thrown. Anyway, thanks for such clear error messages! BINARY-LENGTH-MAP reference doc uses seemingly undefined function PITCH-VECTOR-MAP LENGTH-ADJUST reference shown output of first example call to LENGTH-ADJUST incorrect (seemingly the value of my-phrase was changed later). There are several similar errors in the documentation -- do you want such cases reported? GEN-LENGTH-CARTESIAN again, shown output of some calls to GEN-LENGTH-CARTESIAN incorrect (possibly the see changed later?) Some functions used in code are reported as undefined by system (downloaded version 1.0.15665) function INTEGER-TO-PITCH used in harmonic-path.rtfd BTW: Calls to the function klangreihen in harmonic-path.rtfd also result in errors. LENGTH-MODIFY If the count argument is not given to LENGTH-MODIFY, then seemingly count is randomised. That should be documented, otherwise it is confusing. In the following example I would otherwise expect always the same result (a 1/16 added to all notes, but not the rests), but notes are randomly modified or not. (length-modify 1/16 '(1/8 1/8 1/8 1/8 -1/8 1/8 1/8 -1/8) :operator 'add :type 'n) Also, the value 'all for :type is not documented (but thanks for your error messages pointing that out!!) VELOCITY-INVERT VELOCITY-INVERT ignores dynamics marks like cresc. It would be useful if (and users may expect that) those are also inverted. If not, then at least the documentation should mention this for completeness. VELOCITY-RETROGRADE VELOCITY-RETROGRADE reverses the order of dynamics, including marks like cresc., but it might make more sense for users if such dynamics are "turned around" in the result, e.g., (f> p) -> (p< f) Of course, this is a bit more tricky :-) Again, if not then the the documentation should perhaps at least mention this for completeness. Quote Link to comment Share on other sites More sharing options...
opmo Posted December 26, 2014 Share Posted December 26, 2014 Thanks for the typos and errors, just doing the corrections :-) Quote Link to comment Share on other sites More sharing options...
opmo Posted December 28, 2014 Share Posted December 28, 2014 The version 1.0.15680 fixed all the docs errors. Quote Link to comment Share on other sites More sharing options...
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.