Skip to content
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

[WIP] Initial version of a QNN lecture. #152

Merged
merged 24 commits into from
Nov 19, 2021
Merged

[WIP] Initial version of a QNN lecture. #152

merged 24 commits into from
Nov 19, 2021

Conversation

magnus-the-collectioner
Copy link
Collaborator

No description provided.

@vvssttkk
Copy link
Member

немного поспешил, ждем когда будет не черновик

@vvssttkk
Copy link
Member

vvssttkk commented Sep 3, 2021

@magnus-the-collectioner как обстоят дела?

@Yorko
Copy link
Collaborator

Yorko commented Sep 6, 2021

ааа, черт, я что-то не так делаю, тащу изменения из других веток – пошел ботать Git.

В-общем, тут я добавил QNN в _toc.yml и внес пару мелких фиксов.

Читается здорово, пока прям ревью рано делать, но некоторые места хотелось бы поподробнее. Может, пример дальше это пояснит

@Yorko
Copy link
Collaborator

Yorko commented Sep 7, 2021

Где-то косяки пошли с мёржами. В частности, правки к лекциям 4, 5 и 10 по питону залиты (см. #139, #140 и #147), но на данный момент их нет в мастере: https://github.com/SemyonSinchenko/qmlcourse/tree/master/qmlcourseRU/book/pythonblock. Так что вернул, и в этом PR они как новые файлы показываются.

@vvssttkk
Copy link
Member

vvssttkk commented Sep 7, 2021

после решения вопросов с #154 лишние лекции тут нужно будет убрать

@Yorko
Copy link
Collaborator

Yorko commented Sep 7, 2021

Как #154 будет вмержен, эти файлы не должны будут в диффе показываться, т.к. будут и в этой ветке, и в мастере. В-общем, Никит, сорян, что в этой ветке устроил эту разборку, надо было сразу в отдельной.

@Yorko
Copy link
Collaborator

Yorko commented Sep 7, 2021

Немного запутался, но норм, теперь лишних файлов в диффе нет

@magnus-the-collectioner
Copy link
Collaborator Author

Если есть какие-то предложения, что конкретно можно попробовать расписать чуть подробнее - напишите, попробую это сделать на днях.

@SemyonSinchenko
Copy link
Collaborator

Если есть какие-то предложения, что конкретно можно попробовать расписать чуть подробнее - напишите, попробую это сделать на днях.

Я там в основном все сделал как коммит-саджешн, ты можешь просто тыкнуть в интерфейсе прямо (коммит саджешн). Вопрос ток по картинкам - скинь мне на них ссылки, я попробую поискать в качестве получше и надо ссылки на них ставить.

Короче просто сам внес мелкие правки.
@magnus-the-collectioner - посмотри тогда.

- По первым двум картинкам - поправил разрешение и добавил ссылки
- Про анзатц добавил маленькую заметочку
- Поправил везде поехавшие списки

Changes to be committed:
- modified:   qmlcourseRU/_bibliography/references.bib
- new file:   qmlcourseRU/_static/qnnblock/layer_cvqnn.png
- new file:   qmlcourseRU/_static/qnnblock/qnntfq2.png
- modified:   qmlcourseRU/_toc.yml
- modified:   qmlcourseRU/book/qnnblock/qnn.md
@vvssttkk
Copy link
Member

@magnus-the-collectioner не забудь пж закончить лекцию

@vvssttkk vvssttkk linked an issue Sep 27, 2021 that may be closed by this pull request
@SemyonSinchenko SemyonSinchenko marked this pull request as ready for review October 13, 2021 08:10
@SemyonSinchenko
Copy link
Collaborator

Перевел в стадию PR

@vvssttkk vvssttkk changed the title Initial version of a QNN lecture. [WIP] Initial version of a QNN lecture. Oct 20, 2021
@vvssttkk
Copy link
Member

доступ к stage лекции – http://54.89.156.216/152_merge/_build/html/book/qnnblock/qnn.html

@meandmytram
Copy link
Collaborator

Посмотрел лекцию, в качестве обзора очень круто получилось, как мне кажется. Подправил пару стилистических деталей.

qmlcourseRU/book/qnnblock/qnn.md Outdated Show resolved Hide resolved
qmlcourseRU/book/qnnblock/qnn.md Outdated Show resolved Hide resolved
qmlcourseRU/book/qnnblock/qnn.md Outdated Show resolved Hide resolved
qmlcourseRU/book/qnnblock/qnn.md Outdated Show resolved Hide resolved
qmlcourseRU/book/qnnblock/qnn.md Outdated Show resolved Hide resolved
@vvssttkk
Copy link
Member

@magnus-the-collectioner посмотришь мои комментарии, а то ты автор и вольный принимать решения что с ними делать?
и у других нет замечаний/предложений, а то охота уже влить в master?

@vvssttkk
Copy link
Member

готов вливать в master, все всё просмотрели?

@vvssttkk vvssttkk merged commit f089c7b into master Nov 19, 2021
@vvssttkk
Copy link
Member

@magnus-the-collectioner спасибо за лекцию

@vvssttkk vvssttkk deleted the lecture/qml branch November 19, 2021 19:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Лекция про квантовые нейросети
6 participants