Fluid.bufchroma help: a few errors

[tremblap]

Love the patch. A few problems, from our emails

1. the text from the doc is cropped (see picture below)
2. "an histogram" should be "a histogram" and "class" should be "object" as I was told by @jamesb93
3. there are bufcompose errors every time it loads a file
4. in the 'more details' we need (like the fluid.chroma~) a way to adapt to varying number of chroma bin. the code is almost there. it will be good to show users how to dismiss the spare channels.

[jamesb93]

1. the text from the doc is cropped (see picture below)

That is solved by changing the layout of the patch now in the linked PR.

2. "an histogram" should be "a histogram" and "class" should be "object" as I was told by @jamesb93

That is a docs issue not a Max issue so will be solved there.

3. there are bufcompose errors every time it loads a file

bug squashed

4. in the 'more details' we need (like the fluid.chroma~) a way to adapt to varying number of chroma bin. the code is almost there. it will be good to show users how to dismiss the spare channels.

AFAICT it will adapt in more details. The below screenshot shows the part of the patch that calculates bin frequencies.

[tremblap]

good for 1-2-3

#4: maybe the 'more details' can be streamlined to not do audio (like what I proposed in the Pd version) as it is 'more advanced' and there are still complicated points to explain (why the empty channels) ?

[tremblap]

also, the current patch doesn't adapt - it should not adapt to the number of channels (as it will always be 19) but to the numchroma, which will need to be passed differently than it is right now. But again if we go for my proposal it is a moot point but this error shows we need to explain the 19 channels of buffer one will get, of which numchroma only are non-zero/used/computed

[jamesb93]

also, the current patch doesn't adapt - it should not adapt to the number of channels (as it will always be 19) but to the numchroma, which will need to be passed differently than it is right now. But again if we go for my proposal it is a moot point but this error shows we need to explain the 19 channels of buffer one will get, of which numchroma only are non-zero/used/computed

I see I see. Let's loop back around to this at some point after the imminent release. I have too much on my plate to contemplate these finer details.

[jamesb93]

Issues fixed in #129