Skip to content

Added functionality to give a description in the progress bar#401

Merged
dfm merged 7 commits into
dfm:mainfrom
arm61:pbar_desc
Aug 6, 2021
Merged

Added functionality to give a description in the progress bar#401
dfm merged 7 commits into
dfm:mainfrom
arm61:pbar_desc

Conversation

@arm61

@arm61 arm61 commented Aug 6, 2021

Copy link
Copy Markdown
Contributor

tqdm allows a description to be given to the progress bar, I find this is helpful in packages where multiple different progress bars are used. This PR allows a description to be added to a emcee progress bar based on a string passed to to the sample function.

@dfm

dfm commented Aug 6, 2021

Copy link
Copy Markdown
Owner

Thanks! What about adding a progress_kwargs dictionary or something to allow other keyword arguments beyond just desc?

@arm61

arm61 commented Aug 6, 2021

Copy link
Copy Markdown
Contributor Author

That is a cleaner way. I have put it in like this, happy to know your thoughts.

@dfm dfm merged commit bf0964a into dfm:main Aug 6, 2021
@dfm

dfm commented Aug 6, 2021

Copy link
Copy Markdown
Owner

Looks good to me! Thanks!!

@arm61

arm61 commented Aug 18, 2021

Copy link
Copy Markdown
Contributor Author

@dfm could I be a pest and ask for a pypi release with this feature? I can't get my package onto pypi currently as it requires this feature and pypi doesn't like direct dependencies to git repos apparently.

@dfm

dfm commented Aug 23, 2021

Copy link
Copy Markdown
Owner

Absolutely! I'll do that now.

@dfm

dfm commented Aug 23, 2021

Copy link
Copy Markdown
Owner

v3.1.1 should be on PyPI shortly (as soon as this finishes running)

@arm61

arm61 commented Aug 23, 2021

Copy link
Copy Markdown
Contributor Author

Amazing! Thanks for your help 🙏🏻

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.

2 participants