Skip to content

feat(APS): add some lecture photos - #163

Open
footbooller wants to merge 4 commits into
MPSU:english_versionfrom
footbooller:english_version
Open

feat(APS): add some lecture photos#163
footbooller wants to merge 4 commits into
MPSU:english_versionfrom
footbooller:english_version

Conversation

@footbooller

Copy link
Copy Markdown

No description provided.

Comment thread .pic/Lectures/02. Instruments/fig_01.jpg

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

видимо ты в drawio поверх накладывал текст, у тебя блок текста с АЛУ перекрывает линию самого алу, плюс этот текст невыровнен. Аналогично для write и right

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тебе лучше убрать фон у текста (но перед этим либо стереть старый текст, либо замазать его белым прямоугольником

Comment thread .pic/Lectures/04. Operations units/fig_06.png
Comment thread .pic/Lectures/04. Operations units/fig_07.png
Comment thread .pic/Lectures/06. RISC-V architecture/fig_05.jpg
Comment thread .pic/Lectures/09. Multicycle processor/fig_02.png
Comment thread .pic/Lectures/09. Multicycle processor/fig_03.png
Comment thread .pic/Lectures/13. Processor architecture classification/fig_01.png
Comment thread .pic/Lectures/14. Interrupt subsystem/fig_14.jpg

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Здесь и далее: слишком мелко, сделай на прозрачном фоне просто, чтоб не перекрывало дробь

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Здесь и далее: все еще мелковато, числа и MR всё ещё гораздо больше, в итоге какая-то длинная палка дроби и мелкий текст, нужно увеличить шрифт либо пока размер не совпадает с MR, либо пока знаменатель не займет всю ширину дроби

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В идеале, текст знаменателя должен занимать всю ширину дроби, а размер MR и чисел и числителя должен быть подогнан под размер текст знаменателя

@footbooller

Copy link
Copy Markdown
Author

пофиксил несколько тредов

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тут как-то мелковат стал текст, в предыдущей версии он был куда читаемей, но были белые пятна, а теперь нет пятен, но текст стал мелким

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В остальном правки норм, закрыл соответствующие треды

@footbooller

Copy link
Copy Markdown
Author

таблицу из Харрисов брал

@HepoH3 HepoH3 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Все хорошо, зарезолвил соответствующие комменты из старых ревью, за исключением коммента далее

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Здесь и далее: все еще мелковато, числа и MR всё ещё гораздо больше, в итоге какая-то длинная палка дроби и мелкий текст, нужно увеличить шрифт либо пока размер не совпадает с MR, либо пока знаменатель не займет всю ширину дроби

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants