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

s4583222 - Stable Diffusion model (Wrong branch fix) #484

Open
wants to merge 32 commits into
base: topic-recognition
Choose a base branch
from

Conversation

s4583222
Copy link

@s4583222 s4583222 commented Nov 9, 2022

Hi all,
My original pull request got deleted when trying to fix the issue of my report being in the wrong branch.
My report was on stable diffusion using the OASIS brain dataset.
My files have been split up into the following

  • predict.py
  • train.py
  • modules.py
  • dataset.py

Thanks,
Tom

Model will now train. Have trained over night and so far has a loss of 0.05
This commit is for debugging, getting the code running on Rangpur, as well as a complete reconstruction of my UNET model. It was too big which resulted in the GPU running out of memory. I am trying a different architecture.
Also Progress for the README report
I had an issued running my code on gpu's. It would run out of memory after about 3 epochs. I have changed up my code and now my code has been training for 11 hours
I trained for 18 hours and found that it wasn't training correctly. I have gone back a written my training and Unet code
I had an issue with my reverse process. I was not feeding back the noised image
I have added in another time to save the model (but to a separate file for a back up). I let it train overnight and it show how crashed, when I tried loading the model it was corrupted. This is because it would've crashed whilst saving the model. I have created a separate instance of it saving the model. But at a different time to the first save. That way if one becomes corrupted I can load up from a 5 epochs ago. Also have fixed my de noising for my predict file. Which can show images of stuff that look similar to brains.
@shakes76
Copy link
Owner

You history looks good, but PR is really late. Can you post link to the original (closed?) PR ASAP?

@shakes76 shakes76 added Ungraded Missing ident, so upload grade when found Diffusion Model labels Nov 20, 2022
@shakes76
Copy link
Owner

Good Practice (Design/Commenting, TF/Torch Usage)

Adequate use and implementation
Good spacing and comments
Header blocks missing -1

Recognition Problem

Solves problem (some poor recons, but difficult problem) -2 +2
Driver Script present
File structure present
Shows Usage & Demo & Visualisation & Data usage
Module present
Commenting
No Data leakage
Difficulty: Hard

Commit Log

Meaningful commit messages could be more descriptive -1
Progressive commits used

Documentation

ReadMe OK, broken links -2
Good Description and Comments
Markdown used PDF submitted

Pull Request

Successful Pull Request (Working Algorithm Delivered on Time in Correct Branch)
No Feedback required
Request Description good

LATE: Waiting for PR link to confirm marks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Diffusion Model Ungraded Missing ident, so upload grade when found
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants