-
-
Notifications
You must be signed in to change notification settings - Fork 60
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
PS3: misprint and one suggestion #56
Comments
You're welcome and thanks! |
Fixed-point unit diagram fixed |
I have one more suggestion. |
The problem is that the reader may not know what the SPU is or how the OS is composed, yet. So I have the impression adding that info so early would make it more difficult to follow the article. |
I don’t think it overcomplicate the article. Moreover, in “3.6.1.2 The Synergistic Processor Unit” you wrote “remember there are seven to choose from”. As we know game developers have only six to choose from. In my opinion it’s better to clarify this point earlier. At this moment the reader only needs to know that of the eight SPU one is disabled, one is performing OS tasks and six are available for games. |
Part comes from PSX-Place (https://www.psx-place.com/threads/playstation-3-architecture-a-practical-analysis-by-rodrigo-copetti.35460/) Another part comes from #56
Part comes from PSX-Place (https://www.psx-place.com/threads/playstation-3-architecture-a-practical-analysis-by-rodrigo-copetti.35460/) Another part comes from #56
I'll still try to find a way to bring the double arrows to the diagrams. |
If I understand correctly, for diagrams you use draw.io. The double arrows are in “General”. |
“Handmade” arrow that looks very similar. Consist from two lines. Styles for them. endArrow=none;html=1;rounded=1;strokeColor=#000000;strokeWidth=1;jumpStyle=none;jumpSize=6;shape=link;width=4; endArrow=none;html=1;rounded=0;strokeColor=#000000;strokeWidth=1;jumpStyle=none;jumpSize=6;elbow=vertical; |
Example: https://images.app.goo.gl/MbaWxQYTocf6AFzT7
Thank you for your articles!
The text was updated successfully, but these errors were encountered: