-
Notifications
You must be signed in to change notification settings - Fork 10
Description
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.