-
Notifications
You must be signed in to change notification settings - Fork 218
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
Improved README.md and added Examples #15
base: master
Are you sure you want to change the base?
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.
Also, remove changes related to GENTRL pipeline image (both pdf and png), as they seem to be unchanged
Co-Authored-By: Daniil Polykovskiy <daniil.polykovskiy@gmail.com>
Hi, I have made all the changes. Please review it. |
Hi, Can you please review my changes and merge the pull request. |
Note a missing comma at setup.py, line 14 as per https://youtu.be/ya3AdrfKYzc?t=2470 |
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.
All the changes incorporated
Hi insilicomedicine,
I have tried updating the Readme file to include more detailed instructions on the installation and running of the model. I have also tried to give some explanation based on my understanding of the model and examples along with some examples of the output images.
Please consider this for the community as I have many colleagues and friends who had hard time running your amazing model.
In the future I am thinking of doing these:
Thank you
I hope you consider my request