-
Notifications
You must be signed in to change notification settings - Fork 11
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
Update Readme and added ipynb file #7
Conversation
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.
@Preetesh21 Great work!
Good inferences on the epoch vs error graphs.
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.
Could you add a comparison of the previous model and this model?
Plot graphs for epoch vs PSNR, epoch vs PC and epoch vs SSIM for both the methods together, so that we can visually analyze it?
Sure. |
Added the Graphs for the Parameters. |
Hi @Preetesh21
This way we can compare the performance of both models for each of the parameters. Since we want to compare their predictive capabilities, we don't need to have the training curve, only the validation curve is required. |
ok sure. |
Done the requested changes. |
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.
Great work @Preetesh21
give @Preetesh21 25 points |
Fixes #5
Update Readme and added ipynb file for the issue #5