Change the way idxs are recalculated for dynesty

• Previously, matching was only performed on the first element of the samples. This performs it over all the samples.
• Applied to dynesty and nestle
• Used explicit for loop to make it clear what is going on

Resolve #200 (closed)

I have a couple of points on this.

1. I'm just reading and not running any code, but it looks like my code has a bug, I think there's a misplaced ")".
2. The version I had shouldn't check the first, although I believe the bug may be causing that. It will still fail though if a single parameter matches to within machine precision.

added 1 commit

• b8f81a9e - Change from a boolean sum to an all statement

Compare with previous version

Feel free to basically ignore my other comments, I think I wasn't making much sense.

added 1 commit

• eef33193 - Change from a boolean sum to an all statement

Compare with previous version

added 1 commit

• 7b2fadbb - Adds ValueError and move functionality to method

Compare with previous version

Okay I added a message to the ValueError and moved it all to a common method

added 1 commit

• debd82c2 - Fix flake8

Compare with previous version

approved this merge request

resolved all discussions

approved this merge request

mentioned in commit cb9c52de

merged

mentioned in commit 0fdeabd4