Rename obtuse functions

DiODeProject / MuMoT

Multiscale Modelling Tool - mathematical modelling without the maths

https://mumot.readthedocs.io/

GNU General Public License v3.0

21 stars 4 forks source link

Rename obtuse functions #110

Closed jarmarshall closed 6 years ago

jarmarshall commented 6 years ago

Here are function names I think are not intuitive:

numSimStateVar
numSimNoiseCorrelations
showNoiseEOM
showNoiseStationarySol

Here are proposals for new names:

integrate
integrateNoise (not sure this is really accurate)
showNoiseEquations
showNoiseSolutions

Also, really showODEs_vKE should actually be replaced by a keyword heuristic / vanKampen in showODEs

Comments on naming welcome...

joefresna commented 6 years ago

integrate might be integrateODEs
integrateNoise might be noiseCorrelations

jarmarshall commented 6 years ago

Remove old functions @tbose1 and also rename classes to follow Pythonic naming convention (e.g. MuMoTbifurcationView not MuMoTBifurcationView

jarmarshall commented 6 years ago

For showODEs default to heuristic (but rename this to massAction)

tbose1 commented 6 years ago

Solved with commit e4af1a5f850c8a03d50fac1df74a7ff58f56f0c7. Implemented everything as discussed. Function/method names will be updated in user manual shortly.