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

Glossary #48

Merged
merged 31 commits into from
Jun 8, 2021
Merged

Glossary #48

merged 31 commits into from
Jun 8, 2021

Conversation

karelin
Copy link
Collaborator

@karelin karelin commented Jun 7, 2021

Подготовил первичный вариант глоссария и примеры ссылок на термины из отдельных глав.

По предложению Семена давайте сольем это в мастер, чтобы было проще всем авторам отдельных глав добавлять термины в глоссарий (минимально термин и его английский вариант).

По использованию глоссария: предлагаю при первом упоминании термина в какой-то главе давать ссылку на глоссарий, пара примеров добавлены в текст и в CONTRIBUTING.md.

Готов к критике и предложениям!

karelin and others added 29 commits April 18, 2021 18:49
Поправила орфографию.
Поправила орфографию и пунктуацию; упростила предложение.
Поправила пунктуацию.
Поправила орфографию и пунктуацию; сократила и отредактировала некоторые предложения в инфостиле: выглядит более компактно и информативно.
Поправила орфографию и пунктуацию; сократила и отредактировала некоторые предложения в инфостиле: выглядит более компактно и информативно.
On branch feature/deployement
Changes to be committed:

- new file:   .github/workflows/book.yml
Вернула -- тире и кавычки ""
Вернула -- тире и кавычки ""
Вернула тире -- и кавычки ""
@SemyonSinchenko
Copy link
Collaborator

@karelin А сможншь добавить алфавит как заголовки 2-го уровня (##) или даже 3-го уровня - тогда они будут справа отображаться и это упростит навигацию.

@Yorko
Copy link
Collaborator

Yorko commented Jun 8, 2021

Пофиксил опечатку (одну) и URLs сделал не просто текстом.

В этом PR много лишнего – изменения подтянулись из других PR/коммитов. К тому же, сейчас конфликт с файлом qmlcourseRU/book/vqcblock/vqc.md UPD: ✅

В целом по глоссарию нет вопросов. Разве что присутствие некоторых терминов без пояснений не приносит пользы, на мой взгляд, а только делает глоссарий длиннее (fSim, iSWAP).

qmlcourseRU/book/glossary.md Show resolved Hide resolved
Собственное значение
Англ. Eigenvalue

Соотношение неопределенности
Copy link
Collaborator

Choose a reason for hiding this comment

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

Тут, может, "принцип неопределенности" сделать базовым используемым вариантом, а "соотношение неопределенности" – опционально?

Copy link
Collaborator

Choose a reason for hiding this comment

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

У ландау вроде соотношение. Давайте пока оставим.

@Yorko
Copy link
Collaborator

Yorko commented Jun 8, 2021

Еще при сборке кидает предупреждения "WARNING: glossary terms must not be separated by empty lines"

@SemyonSinchenko
Copy link
Collaborator

Еще при сборке кидает предупреждения "WARNING: glossary terms must not be separated by empty lines"

Если убрать пустые линии, то просто WARNING станет другим :)

image

@karelin
Copy link
Collaborator Author

karelin commented Jun 8, 2021

@SemyonSinchenko Разбиение на секции по буквам алфавита проверил, могу сегодня сделать.

@Yorko К сожалению у меня при сборке появляется куча левых сообщений, ворнинг просто потерялся. По набору терминов - можно уточнить полиси, и объяснять только наиболее важные.

@SemyonSinchenko
Copy link
Collaborator

Разбиение на секции по буквам алфавита проверил, могу сегодня сделать.

Давай отдельно уже. По эджайлу чтобы :)

@SemyonSinchenko SemyonSinchenko merged commit eb87243 into master Jun 8, 2021
@karelin
Copy link
Collaborator Author

karelin commented Jun 8, 2021

Давай отдельно уже. По эджайлу чтобы :)

Хорошо, тогда продолжаю в той же ветке?

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.

5 participants