Skip to content

Add progress meter functionality#202

Open
ASKabalan wants to merge 4 commits intopatrick-kidger:mainfrom
ASKabalan:add-progress
Open

Add progress meter functionality#202
ASKabalan wants to merge 4 commits intopatrick-kidger:mainfrom
ASKabalan:add-progress

Conversation

@ASKabalan
Copy link
Copy Markdown

Answers #134

@ASKabalan
Copy link
Copy Markdown
Author

BTW I just copy pasted the progress bar from Diffrax

Let me know if you are interested in a more general progress bar implementation for JAX that supports multiple bars with vmap and shardmap => https://github.com/ASKabalan/jbar

@ASKabalan
Copy link
Copy Markdown
Author

@patrick-kidger I fixed the mkdocs error

the workflow should work now

Thank you

Copy link
Copy Markdown
Owner

@patrick-kidger patrick-kidger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR! Sorry that this took me so long to get to 😅

So this kind of functionality is something I would put in equinox.internal: it's not large enough for me to spin it out into its own library (as you've done!), but I also want to avoid duplicating it amongst the libraries that I have.

Would you be interested in opening a PR adding this to Equinox (+then pointing Optimistix at it + refactoring Diffrax to use it)? If so then I'd be very happy to support progress bar functionality in Optimistix. :)

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