expose dynamical_system's module names#16
expose dynamical_system's module names#16leavesandlemmas wants to merge 2 commits intobiocro:mainfrom
Conversation
|
Back in the olden days, the BioCro development team had some discussions about methods like these. Justin and Scott R. were of the opinion that a class shouldn't have a getter method that simply returns argument values that were passed to the class constructor. They are unnecessary because in principle, the user should already know these values. I used to think otherwise, but I was convinced by this line of reasoning. In this particular case, the user must already know the names of the direct and differential modules, since they must have passed these to the dynamical system constructor. Just wondering -- what's the reason for adding these getter functions? |
|
The reason is to serve as an example of what I talk about it #15 The getter functions are simple to implement. I don't think that this particular getter function is particularly useful. But I am not sold on the general principle you describe. Consider an counterexample: the But clearly it is useful because a function that accepts a vector might need to know its size, even though the function didn't create it. Internally, So getter functions for BioCro |
Added API methods to
dynamical_system. I am doing a pull request because I can't publish branches onframeworkand I have a branch on biocro/biocro that doesn't compile unless these methods are added.