Phase coherence broken when SendContainer comes before Sampler in the module tree

[Orvillain]

No one has any thoughts on this? Or am I just being a n00b and discovering an aspect of HISE that everyone else already knows about? Seems like a bug to me.

[d.healey]

@Orvillain said in Phase coherence broken when SendContainer comes before Sampler in the module tree:

So if the Send Container comes after the Sample

I'm pretty sure in the docs it mentions that the send container must be after.

[d.healey]

Actually it's not in the docs, it's in this post from Christoph - should be added to the docs though.

[clevername27]

@Orvillain what, on earth, are you doing with 10 kick-sends?

[Christoph Hart]

should be added to the docs though.

Yes and / or add even a warning on the module interface if you connect a send FX to a container that comes before it as it's a very subtle issue that would slip through easily.

[Orvillain]

@Christoph-Hart said in Phase coherence broken when SendContainer comes before Sampler in the module tree:

should be added to the docs though.

Yes and / or add even a warning on the module interface if you connect a send FX to a container that comes before it as it's a very subtle issue that would slip through easily.

Or just modify the code so that when you add a send container, it always gets added to the end of the module it is being added to?

[d.healey]

@Orvillain said in Phase coherence broken when SendContainer comes before Sampler in the module tree:

added to the end of the module it is being added to?

You don't necessarily always want this so I wouldn't make it that strict.

[Christoph Hart]

You don't necessarily always want this so I wouldn't make it that strict.

Actually I can‘t think of a use case where having the container before the source would be useful - usually one could argue that this can be used to implement feedback loops like you do with the scriptnode send and receive nodes but this should be done in scriptnode and the send container on the HISE module level‘s prime use case is signal routing.

[Orvillain]

@d-healey said in Phase coherence broken when SendContainer comes before Sampler in the module tree:

@Orvillain said in Phase coherence broken when SendContainer comes before Sampler in the module tree:

added to the end of the module it is being added to?

You don't necessarily always want this so I wouldn't make it that strict.

I don't think it should be enforced necessarily. Just as a serving suggestion. You could still place the module wherever you want using the Builder.

In any case, this should definitely be documented. I'd be happy to add something to the documentation myself, if it is possible.

[d.healey]

@Christoph-Hart said in Phase coherence broken when SendContainer comes before Sampler in the module tree:

Actually I can‘t think of a use case where having the container before the source would be useful

You may want the container to be after the source but not want it to be at the end of the chain. I can't think of a particular use case, but it may come up in the future.