Corrected deprecated comments and followed best practices#8
Corrected deprecated comments and followed best practices#8Francisco-G-P wants to merge 3 commits intoABarpanda:mainfrom
Conversation
|
Hi, @ABarpanda I've submitted the reviewed version of the Animegen files! Please let me know if there are changes to be made. |
ABarpanda
left a comment
There was a problem hiding this comment.
The placement of start = timeit.default_timer() in main.py was intentional.
When we import scene_split, some part of the program is run, which delays the execution of main.py. Since that is also included in the time we have to wait for the program to run, start must be placed before imports.
|
I understand, sorry for the inconvenience. I've applied the necessary change to main.py. |
ABarpanda
left a comment
There was a problem hiding this comment.
Hey @Francisco-G-P , the import slideshow was also intentionally placed at the end.
Please refer to #3 for more detail.
|
After reviewing Issue #3 I got why it was placed at the end of the file, thanks! I've applied the necessary change to main.py. |
|
Hi! @ABarpanda I've applied the last two changes to the Animegen files! Please let me know if there are any more changes needed to be made. |
While the main task was to eliminate some deprecated comments, some more were added in specific areas where a brief explanation was needed. Most of the deleted comments were
printstatements for what seem where early tests.At the same time, comments that were added or restructured were done following best practices. In main.py the
startvariable was moved away from the imports and in text_to_img.pyfrom PIL import Imagewas deleted as its use was commented out (that comment was deleted as well).