Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update blobs docs (#352) #464

Merged
merged 1 commit into from
May 23, 2023
Merged

Update blobs docs (#352) #464

merged 1 commit into from
May 23, 2023

Conversation

znicholls
Copy link
Contributor

@znicholls znicholls commented Mar 15, 2023

Firstly, massive thanks to @dfm for all your efforts with emcee, it is an amazing tool and community and has made a massive difference to my (and I'm sure many others') research.

Second, thanks to @steven-murray and @lboogaard for your comments in #352. I was also confused by this and your observations were super helpful.

I think this MR will close #352. It is a pretty basic update to the blobs docs to fix a couple of typos and add some more context which I would have found helpful in hindsight and addresses the things brought up in #352 I believe.

The blobs docs were found to be a bit confusing by some users (including me).
This merge request updates them following the discussion in dfm#352 to clarify
how this interface can be used and the intent behind it.
@dfm dfm merged commit 5471ce2 into dfm:main May 23, 2023
@dfm
Copy link
Owner

dfm commented May 23, 2023

Thank you!!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Blobs example a bit confusing
2 participants