-
Notifications
You must be signed in to change notification settings - Fork 100
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
Visualize ControlNet results using Weights & Biases Tables #156
Comments
cc: @sayakpaul |
I am okay with the idea. Could you elaborate what are advantages of using |
With |
Thanks for providing the additional context! Having used tables before, I can definitely see where you are coming from! That said, we want our scripts to be minimal in terms of the features we support so that they are easily customizable. Do you have a (git) patch view of the changes that might be needed here to incorporate tables?
|
Hi @sayakpaul I have the changes on my fork here. |
That is a bit hard for me to look through. Could you maybe try to do a |
how do you compare different runs? |
On your dashboard, tables from different runs are actually concatenated in the same table if they are logged under the same key. The best way to compare against different runs, therefore, is to add an additional column called Note that I prefer to log |
Hi, @sayakpaul @yiyixuxu |
@soumik12345 do you have an example that I can play with? The main concern is that we want it as easy to use as possible. Assume a participant who never heard of Weights & biases before; our goal is to make sure:
So I'm not in favor of adding a fancier logging feature to our example training script that will make the code harder to read and may require additional steps to interact on the dashboard. However, if you want to write a separate inference script with this, we are happy to link it :) |
Add optional support for visualization of ControlNet results using
wandb.Table
intrain_controlnet_flax.py
. This can be used for summarizing results during training and inference.The text was updated successfully, but these errors were encountered: