BiologicalRecordsCentre / sparta

Species Presence/Absence R Trends Analyses
http://biologicalrecordscentre.github.io/sparta/index.html
MIT License
21 stars 24 forks source link

occDetBUGScode() seems to be redundant #80

Open drnickisaac opened 6 years ago

drnickisaac commented 6 years ago

This function returns the original sparta or base version of the occupancy model BUGS code. I think it's been made redundant by the metacoding functions getModelFile, getObsModel etc. It's called only once, to compare the file produced by the metacoding with the original. I believe it's no longer possible for them to match, in which case the warning on line 486 of occDetFunc is also redundant. However, chatting with @AugustT I note that meta-coding is only used for some parts of the model assembly - much of it is still read directly from text files.

drnickisaac commented 5 years ago

I just checked this. At present it is used only to compare against the chosen model. However, as we move towards a fully meta-coding paradigm for the BUGS code, we will want to keep this function as the model here is "classic sparta", i.e. the same as used in my sims paper.