-
Notifications
You must be signed in to change notification settings - Fork 1
refactor1 #202
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
base: main
Are you sure you want to change the base?
Conversation
|
⚙️ Codegen is reviewing the Pull Request |
| deletes segments of the video so they can be overwritten afterwards, and Moshy, | ||
| which overwrites these empty segments with information copied from random other | ||
| parts of the video. This process gives the impression that the visual | ||
| love sucks |
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.
The line 'love sucks' (line 33 on the right side) seems to be out of context. It's recommended to maintain a professional tone in the project documentation. Please revise this line to ensure consistency and relevancy within the project's README.
| be seen from both sides. Between the screens are several rotating computer | ||
| chairs that invite the audience to sit. | ||
|
|
||
| To represent the protagonist's loss of memory, the work will slowly "die" over |
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.
The removal of the detailed explanation (left side, lines 35 to 45) of how the project handles the protagonist's memory loss seems problematic. This information is valuable and introduces a significant aspect of the project. Consider adding it back or provide a revised version if the original content was incorrect.
| to Johnny in the narrative when he loses his own memories due to the overload of | ||
| information in his brain. | ||
|
|
||
| it really does sucks |
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.
A similar issue is noticed with the line 'it really does suck' (line 54 on the right side). It's unclear what this statement is referring to, and it seems irrelevant to the project as described in the README. Please revise or remove line 54 as it doesn't provide meaningful content.
No description provided.