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

parser does not detect MODE kWave in LfoC #7

Closed
zappfinger opened this issue Mar 4, 2018 · 7 comments
Closed

parser does not detect MODE kWave in LfoC #7

zappfinger opened this issue Mar 4, 2018 · 7 comments

Comments

@zappfinger
Copy link
Collaborator

The wave selection in LfoC is not a parameter, but a mode (according to ModelDef.xml). It is not picked up by the parser

@gleb812
Copy link
Owner

gleb812 commented Mar 4, 2018

Got it! In process.

@ghost
Copy link

ghost commented Mar 4, 2018

OK, I did this. Example of the generated code is bellow. Also added a notion to the tutorial: https://github.com/gleb812/pch2csd/wiki/Making-a-new-module

; --------------------
; VOICE AREA
instr 1
; Module    Parameters     Modes    Inlets      Outlets
LfoC        62.9,0,4,1,1,  0,       1,                0
LfoC        59.4,0,4,1,1,  1,       1,                0
LfoC        56.0,0,4,1,1,  2,       1,                0
LfoC        52.9,0,4,1,1,  3,       1,                0
LfoC        49.9,0,4,1,1,  4,       1,                0
LfoC        47.1,0,4,1,1,  5,       1,                0
LfoC        44.5,0,4,1,1,  6,       1,                0
LfoC        42.0,0,4,1,1,  7,       1,                0
endin

@ghost
Copy link

ghost commented Mar 4, 2018

17da58f

@ghost ghost closed this as completed Mar 4, 2018
@zappfinger
Copy link
Collaborator Author

zappfinger commented Mar 4, 2018 via email

@ghost
Copy link

ghost commented Mar 4, 2018

@zappfinger Did you include “modes” as arguments to the ;@ args annotation? I made modes special and they don’t need to be included there. You only need to modify the opcode definition to accomodate modes --- they will be included as typical arguments.

E.g. if you have your annotation like this: ;@ args kkk,k,k and you have two mode variables, then your opcode annotation should be like this: opcode SomeName, 0, iiiiiii (seven i’s).

@ghost
Copy link

ghost commented Mar 4, 2018

The motivation for this was to not have to include ;@ map annotation for modes. Do you think this solution is counter-intuitive?

@zappfinger
Copy link
Collaborator Author

zappfinger commented Mar 4, 2018 via email

This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants