christianparobek / skeleSim

Null models, performance testing, and power estimation with population simulations
3 stars 6 forks source link

Simulator slot change #8

Closed EricArcher closed 9 years ago

EricArcher commented 9 years ago

From Sean: In skeleSim.classes.R, the slot simulator, maybe we should change this from "c" and "f" to "fsc" and "rms" to match the two simulators we are using? This would make the program more open ended in the future and seems like a good change for now?

EricArcher commented 9 years ago

Sounds like a good idea. The "c" and "f" came from the "coalescent" and "forward-time" choices the user made in the initial "guidance" series of questions. We can retain that maybe in a "@simulator.type" slot and specify the "@simulator" slot to be a three-character identifier like "fsc" and "rms".