-
Notifications
You must be signed in to change notification settings - Fork 8
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
Add example of optimizing hyperparameters for handwriting recognition #57
base: main
Are you sure you want to change the base?
Conversation
A detailed description of the usage.
Fix typos
Fix typos
Fix typos
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- workflow.py and label.py seem unnecessary, especially when functions are refactored, move to project.py.
I did not have an opportunity to view the VAE code any any depth currently, but I will when these changes are addressed.
projects/flow.pytorch.HyperparameterOptimize/source/workflow.py
Outdated
Show resolved
Hide resolved
projects/flow.pytorch.HyperparameterOptimize/source/workflow.py
Outdated
Show resolved
Hide resolved
plt.figure(figsize=(12, 10)) | ||
plt.scatter(z1_arr, z2_arr, c=colormap(c_arr), s=5) | ||
plt.colorbar(sm, ticks=np.linspace(0, 10, 10, endpoint=False), label="Ground truth") | ||
plt.xlabel(r"$z_1$", fontsize=25) | ||
plt.ylabel(r"$z_2$", fontsize=25) | ||
plt.title(title_name, fontsize=35) | ||
plt.savefig(job.fn("latent.jpg")) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use object oriented interface.
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks to be in good shape, there appear to be CI issues and there is one small thing that I thought I should mention.
PyTorch is not yet available and won't until pytorch/pytorch#112882 is merged, so we will have to wait to merge this. |
Thank you @shihkual for this addition for our poster at MIDAS!