Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Token for multichannel control stream / list #14

Open
jamesb93 opened this issue Oct 26, 2021 · 9 comments
Open

Token for multichannel control stream / list #14

jamesb93 opened this issue Oct 26, 2021 · 9 comments
Assignees
Labels
enhancement New feature or request
Milestone

Comments

@jamesb93
Copy link
Member

In the max reference and supercollider documentation there is reference to CCE specific terminology. This should be managed with a new token so that the CCE exchanges the words.

For example, from pitch~

The process will return a multichannel control steam with [pitch, confidence] values

@jamesb93 jamesb93 added documentation Improvements or additions to documentation enhancement New feature or request labels Oct 26, 2021
@weefuzzy
Copy link
Member

Sounds good – were you planning to PR this, or do you need input from me?

@jamesb93
Copy link
Member Author

I'm flagging it as I go. I'm happy to PR it at some point - I think I should be able to bash my way through implementing it.

@jamesb93
Copy link
Member Author

  1. Multichannel control stream / list
  2. Camelcase-d names of arguments/methods

@tremblap tremblap added this to the beta5 milestone Jan 27, 2022
@jamesb93 jamesb93 assigned jamesb93 and unassigned tedmoore, jamesb93 and weefuzzy Jan 28, 2022
@jamesb93 jamesb93 removed the documentation Improvements or additions to documentation label Jan 28, 2022
@tedmoore
Copy link
Member

tedmoore commented Feb 1, 2022

Is it true that this will be the same implementation as |buffer| is currently?
Can I add to the list a token for 'dictionary' as well so that we can link to it different CCEs.

@jamesb93
Copy link
Member Author

jamesb93 commented Feb 7, 2022

Is it true that this will be the same implementation as |buffer| is currently? Can I add to the list a token for 'dictionary' as well so that we can link to it different CCEs.

Do you need help with this @tedmoore? You are more expert than me in all things docs right now but I'm happy to take this on.

@weefuzzy
Copy link
Member

weefuzzy commented Feb 7, 2022

Is it true that this will be the same implementation as |buffer| is currently? Can I add to the list a token for 'dictionary' as well so that we can link to it different CCEs.

It could be. My only gripe with the substitution syntax is that I haven't found an elegant way of adding it to docutils programatically and have been appending the rst command it to the input string instead, which makes me sad. In any case, clearly the syntax should be the same as for buffer.

@tedmoore
Copy link
Member

tedmoore commented Feb 7, 2022

@james It sounded like you were planning to the to control stream / list one and I was thinking, while you're in there doing that, maybe add a dictionary one as well!? This is a lower priority for me, I was just adding it to the "list" if someone was going to dig and and go after it.

@tremblap
Copy link
Member

tremblap commented Feb 7, 2022

dictionary indeed will be text object in pd - when they exist that is, as not many objects have used them so far - so having a token is a good idea.

@jamesb93
Copy link
Member Author

jamesb93 commented Feb 7, 2022

Okay, I'll attack this in one go then

@tedmoore tedmoore modified the milestones: beta5, beta6 Feb 14, 2022
@jamesb93 jamesb93 modified the milestones: beta6, Version 1.0 Mar 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants