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

validation local blockwise: in progress #294

Closed
wants to merge 29 commits into from
Closed

Conversation

mzouink
Copy link
Member

@mzouink mzouink commented Sep 23, 2024

No description provided.

mzouink and others added 29 commits September 23, 2024 17:36
There appear to be some python formatting errors in
4583723. This pull request
uses the [psf/black](https://github.com/psf/black) formatter to fix
these issues.
There appear to be some python formatting errors in
d899c63. This pull request
uses the [psf/black](https://github.com/psf/black) formatter to fix
these issues.
There appear to be some python formatting errors in
525f2fe. This pull request
uses the [psf/black](https://github.com/psf/black) formatter to fix
these issues.
Adds a simple tutorial start-to-finish tutorial for DaCapo.
Still a few things left to resolve before its ready for merging:

- [ ] Loss behaving strangely after the first validation (temporary
solution involves giving an extra singleton dimension to the provided
data)
- [ ] Validation fails (we want to show validation loss/scores,
validation outputs across iterations, and how to finally take the best
iteration and apply it with the post processing to a volume manually)
- [ ] stdout too verbose
- [ ] plotting functions should be built into `DaCapo` (there are
plotting functions, not sure how to include bokeh plots so I didn't dive
too deep into this)
There appear to be some python formatting errors in
3177b1c. This pull request
uses the [psf/black](https://github.com/psf/black) formatter to fix
these issues.
@mzouink mzouink closed this Oct 23, 2024
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.

3 participants