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
to be valid. However, the comments imply that this should not be allowed:
There should be one response per stage.
In addition, this also means that the Stage does not have input and output units, which are important in understanding the overall response.
I believe the minOccurs attributes are leftover from an earlier version and should be removed, disallowing a Stage with only a StageGain.
For cases where the stage really is only a gain, there should still be a placeholder, like a unity Coefficients filter to provide units and so it is clear what is meant.
The text was updated successfully, but these errors were encountered:
Currently the filters (PolesZeros, etc) have minOccurs=0 on each within ResponseStageType. This allows a Stage like:
to be valid. However, the comments imply that this should not be allowed:
In addition, this also means that the Stage does not have input and output units, which are important in understanding the overall response.
I believe the minOccurs attributes are leftover from an earlier version and should be removed, disallowing a Stage with only a StageGain.
For cases where the stage really is only a gain, there should still be a placeholder, like a unity Coefficients filter to provide units and so it is clear what is meant.
The text was updated successfully, but these errors were encountered: