You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Whether process() is called for an AudioWorkletNode should depend on whether the active source flag is true and whether the AudioNodes connected to inputs are actively processing
#2574
The "rendering a render quantum" steps steps appear to keep calling process() for all AudioWorkletNodes that are "still alive" and have [[callable process]] true.
"still alive" is not defined, but I don't think it is necessary either.
It's tempting to simply replace "still alive" with "actively processing", but this would be problematic due to the dependency of "actively processing" state on the "in a cycle" state, which has not yet been determined at this point in the algorithm.
Perhaps a more carefully placed check on "actively processing" in the "rendering a render quantum" steps would be sufficient to address this.
The text was updated successfully, but these errors were encountered:
The thing I'm not sure is clearly specified is the silent output for nodes that are not actively processing. Would it be worthwhile explicitly "muting" nodes that are not actively processing, to clarify that their last buffer made available for reading is not used? i.e. instead of changing step 1 of "Visiting a node", replace step 5 "Add node to the beginning of ordered node list" with
5. If node is not actively processing, then mute node.
6. Else, add node to the beginning of ordered node list.
While checking the [[callable process]] behavior, I noticed that 4.4.11 "Set [[callable process]] to true" seems out of place. [[callable process]] is initialized true in the constructor, and we don't want it set to true again after "processorerror" has been triggered.
Looks like 4.4.11 can be removed?
The "rendering a render quantum" steps steps appear to keep calling
process()
for allAudioWorkletNode
s that are "still alive" and have [[callable process]] true.The only place that I see the active source flag read is in the definition of actively processing.
"still alive" is not defined, but I don't think it is necessary either.
It's tempting to simply replace "still alive" with "actively processing", but this would be problematic due to the dependency of "actively processing" state on the "in a cycle" state, which has not yet been determined at this point in the algorithm.
Perhaps a more carefully placed check on "actively processing" in the "rendering a render quantum" steps would be sufficient to address this.
The text was updated successfully, but these errors were encountered: