Daniel Turek

Noting this code dates back to the **initial commit of the nimble package** (>9 years ago), and hence the code itself actually predates that commit. This is clearly very old...

Also includes relics such as `identityMatrix`: https://github.com/nimble-dev/nimble/blob/02cffc9fc107a5a1c8f2fd5d938d7558b0c0c184/packages/nimble/R/NF_utils.R#L187

@perrydv It seems reasonable to add this `buildDerivs` argument to `readBUGSmodel`. I'd be happy to make this quick change, but let me know what you think.

@paciorek Agreed, thank you.

@perrydv Yes, I expanded the scope of this PR (after creating the title) to also add the `checkADsupportForDistribution` method for models. I do think this would be a useful addition....

@perrydv Your suggestion does seem to make sense, having this information instead be native to our (existing) distribution info. If I understand, then: - Core distribution definitions (in `distributions_inputList.R`) would...

@perrydv This seems like hugely useful functionality, which I'm quite excited to see. Do you have any code which demonstrates use? It doesn't have to do anything useful, but only...

@perrydv If this is included in the release, my initial reaction is to keep the default as `allowNFinModels=FALSE` for the time being. I think there are some reasonable questions that...

I tend to agree, sounds good.

@perrydv 1. Yes, I'd be happy to see this be **enabled** by default. 2. I'm of the reasonably strong opinion to *not* include any `_dummy` suffixes. Respectfully, I don't see...