Dan Foreman-Mackey
Dan Foreman-Mackey
@Thalos12: Thanks for this!! I'll need a couple of days to get to this, but as a zeroth order comment: how hard do you think it would be to add...
Awesome! Yes - please add some comments about versions to the docstring and tutorial notebook. For the unit test: no stress! I think it'll be tricky to write exactly the...
@Thalos12: Thanks for the detailed code! I'm not sure if I have too much to suggest here because this isn't really a supported use case for this backend and it...
@Thalos12: great! I'd be happy to review such a PR!
@Thalos12: Thanks for looking into this in such detail! Yes - I would be very happy to have this implemented. Can you also include a tutorial in the PR? I'm...
This probably means that there's a bug in your probability function. Please provide more details and try to put together a minimal example of a code snippet that can be...
The first is a bug that I need to fix - thanks for reminding me, but the second is actually fine... take a look at the master docs here: https://emcee.readthedocs.io/en/latest/tutorials/autocorr/...
The problem is with the `return -np.inf` line. That line needs to return the same number of blobs. I *think* it would work fine to just return 4 `None`s, but...
There isn't a worked example anywhere, but I'd recommend checking out the `GaussianMove` source code because it should give you an idea of how it's all supposed to work: https://github.com/dfm/emcee/blob/main/src/emcee/moves/gaussian.py
Yes, it is tested.