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
Some of the examples and support materials throughout the repository refer to ensemble.calculateVolitions() (plural) instead of ensemble.calculateVolition() (singular). The singular name is the actual name of this function in the current version of Ensemble. Either we should update all our materials to refer to the current function name consistently, or we should change the name of this function to ensemble.calculateVolitions() and update all the other places we refer to this function with the new name.
Personally I'm in favor of the latter course of action. I think it's weird and confusing that this function name refers to "volition" (singular) when the thing it's actually doing is calculating and returning all of the volitions (plural). But either way, the inconsistency should be resolved.
The text was updated successfully, but these errors were encountered:
Some of the examples and support materials throughout the repository refer to
ensemble.calculateVolitions()
(plural) instead ofensemble.calculateVolition()
(singular). The singular name is the actual name of this function in the current version of Ensemble. Either we should update all our materials to refer to the current function name consistently, or we should change the name of this function toensemble.calculateVolitions()
and update all the other places we refer to this function with the new name.Personally I'm in favor of the latter course of action. I think it's weird and confusing that this function name refers to "volition" (singular) when the thing it's actually doing is calculating and returning all of the volitions (plural). But either way, the inconsistency should be resolved.
The text was updated successfully, but these errors were encountered: