Fix markup errors in and add to Sound.Tidal.UI documentation.#992
Fix markup errors in and add to Sound.Tidal.UI documentation.#992yaxu merged 7 commits intotidalcycles:mainfrom
Conversation
This patch: - adds a module header that appears in the Haddocks - ports some function descriptions from the main reference documentation - explicates undocumented functions (the ones I could figure out, anyway) - engages in minimal reordering of functions for consistency's stake - fixes a lot of errors resulting from Markdown/Haddock inconsistencies There should be no observable change in behavior, as no code has actually changed.
|
Here is a PDF of what the documentation looks like with this patch: Sound.Tidal.UI.pdf. |
|
Would a maintainer mind approving the workflows here so this can go green? |
matthewkaney
left a comment
There was a problem hiding this comment.
Thank you so much for this! I'm interested in more overlap between the Haddock docs and the tidalcycles.org docs, so this is a great step in that direction.
(As an aside, I see you're in Queens—if you haven't done anything with livecode.nyc, you totally should!)
src/Sound/Tidal/UI.hs
Outdated
| 'degradeBy' and '_degradeBy'). These functions take plain old data types, | ||
| not 'Pattern's, and are mainly useful if you're getting errors about | ||
| ambiguous types. In some cases, they may be more efficient than their | ||
| equivalents that take 'Pattern' values. |
There was a problem hiding this comment.
I'd always assumed that the underscore variants were essentially internal implementations and not intended for public use. If that's the case, then we shouldn't be encouraging their use, and should eventually exclude them from export lists. @yaxu, is that right, or am I wrong and these variants should be considered part of the public API?
Are there cases where you've run into ambiguous type errors?
There was a problem hiding this comment.
If memory serves, I once had enabled perhaps too many {-# LANGUAGE #-} pragmas in the Tidal REPL and got out of it by using an underscore variant. I’m happy to elide this section and just replace it with “the underscore variants are private,” if @yaxu thinks that’s appropriate.
There was a problem hiding this comment.
Hm well I think they shouldn't be totally private, as they're useful in writing custom functions. I would say something like, the underscore variants generally only accept plain values and not patterns apart from their final argument, and are intended mainly for those developing or extending tidal.
There was a problem hiding this comment.
@yaxu Gotcha. I’ve changed the wording to reflect this.
|
Thanks for this and apologies for the long delay ! |
Hey, folks 👋🏻 I’ve been really loving my TidalCycles experience, and wanted to give something back. Since I’m an experienced Haskeller, I tend to use the Haddocks as my primary source of truth, so I’d like to give them a little more love and port over/write documentation when necessary.
This patch:
There should be no observable change in behavior, as no code has actually changed. I hope to have time to renovate some of the other modules.
Other things I could have done to this module, and would be happy to do now or in a follow-up patch:
{-# DEPRECATED #-}, possibly setting the stage for their removal