nimble icon indicating copy to clipboard operation
nimble copied to clipboard

Remove compareMCMCs

Open perrydv opened this issue 1 year ago • 2 comments

I think we could remove the placeholder for the original compareMCMCs (just giving a message) that is now deprecated in favor of package compareMCMCs (which will get an update soon).

perrydv avatar Apr 21 '24 00:04 perrydv

@perrydv Agreed.

danielturek avatar Apr 21 '24 10:04 danielturek

Yes. I'll just put it on my to-do list and take care of with other minor things.

paciorek avatar Apr 22 '24 14:04 paciorek

Working on this now.

@danielturek it appears on initial glance that the only use case for the returnComponents argument to readBUGSmodel was the now defunct MCMCsuite. What do you think about me removing that argument?

paciorek avatar May 18 '24 17:05 paciorek

@paciorek I, personally, think this returnComponents option is useful to have. But that said, if the momentum is to clean up and remove it, I would understand.

danielturek avatar May 19 '24 11:05 danielturek

I'm happy to leave it.

Taking out the placeholders is done on devel, so closing this.

paciorek avatar May 20 '24 15:05 paciorek