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

RU: draft translation of site/ru/beta/guide/effective_tf2.md #651

Open
wants to merge 14 commits into
base: master
from

Conversation

Projects
None yet
7 participants
@stabuev
Copy link
Contributor

commented Jun 9, 2019

also some typo fixes at _index.yaml

@tfdocsbot

This comment has been minimized.

Copy link
Collaborator

commented Jun 9, 2019

Reviewers added, please take a look.
@0101011

When your review is finished, approve the pull request or include "LGTM" in your comment.

@googlebot googlebot added the cla: yes label Jun 9, 2019

@stabuev stabuev changed the title RU: draft translation of site/ru/beta/guide/ RU: draft translation of site/ru/beta/guide/effective_tf2.md Jun 9, 2019

stabuev added some commits Jun 9, 2019

@stabuev stabuev requested a review from brilee as a code owner Jun 9, 2019

stabuev added some commits Jun 9, 2019

same sentence twice
deleted second one
@@ -558,8 +558,7 @@
"* Collect layer parameters in `__init__`.\n",
"* Build the variables in `build`.\n",
"* Execute the calculations in `call`, and return the result.\n",
"\n",
"The `tf.variable_scope` is essentially a layer of its own. So rewrite it as a `tf.keras.layers.Layer`. See [the guide](keras/custom_layers_and_models.ipynb) for details."

This comment has been minimized.

Copy link
@lamberta

lamberta Jun 10, 2019

Member

Keep this, please

This comment has been minimized.

Copy link
@stabuev

stabuev Jun 11, 2019

Author Contributor

I've reverted this commit but isn't it duplicate?

@@ -0,0 +1,1621 @@
{

This comment has been minimized.

Copy link
@lamberta

lamberta Jun 10, 2019

Member

stage: https://colab.research.google.com/github/stabuev/docs/blob/master/site/ru/beta/migration_guide.ipynb

Getting a JSON parse error: Unexpected token / in JSON at position 22156

This comment has been minimized.

Copy link
@stabuev

stabuev Jun 11, 2019

Author Contributor

Fixed this

stabuev added some commits Jun 11, 2019

Revert "same sentence twice"
This reverts commit 9d6388b.
@lamberta

This comment has been minimized.

Copy link
Member

commented Jun 11, 2019

Thank you, Soslan.
For new (large) files, can you create separate pull requests/branches? This will make it easier for reviewers to go through them. (Finding reviewers is always a difficult part of the community translation process.)

@stabuev

This comment has been minimized.

Copy link
Contributor Author

commented Jun 11, 2019

Sure, do this for next translations

@stabuev

This comment has been minimized.

Copy link
Contributor Author

commented Jun 18, 2019

Maybe it is better to split this pull request to several ones for faster review?

@lamberta

This comment has been minimized.

Copy link
Member

commented Jun 18, 2019

Unfortunately, we just don't have many community reviewers for Russian.
Perhaps you can request on the docs-ru@tensorflow.org list? https://groups.google.com/a/tensorflow.org/forum/#!forum/docs-ru

Otherwise, I'll give it some time and merge it---perhaps double-check with Google Translate or something like that. Thanks.

@@ -22,7 +22,7 @@ landing_page:
<code class="devsite-terminal">pip install tensorflow==2.0.0-beta0</code>
</pre>
<p>
Смотрите <a href="/install/gpu">руководство GPU</a> для CUDA®-enabled карт. The
Смотрите <a href="/install/gpu">руководство GPU</a> для CUDA®-enabled карт.

This comment has been minimized.

Copy link
@0101011

0101011 Jun 18, 2019

Contributor

...для карт с поддержкой CUDA.

@@ -37,7 +37,7 @@ landing_page:
<a target="_blank" class="external" href="https://colab.research.google.com/github/tensorflow/docs/blob/master/site/en/r2/tutorials/quickstart/beginner.ipynb">пример “Hello World”</a>,
затем прочитайте руководства и учебники чтобы узнать больше.</p>
<h3>Для экпертов</h3>
The Subclassing API представляет define-by-run интерфейс для продвинутых исследований.
Subclassing API представляет define-by-run интерфейс для продвинутых исследований.

This comment has been minimized.

Copy link
@0101011

0101011 Jun 18, 2019

Contributor

"Subclassing API представляет собой динамический define-by-run интерфейс для продвинутых исследований."

Я думаю следует добавить "динамический" как ключевое определяющее слово для подхода define-by-run, более подробно: https://github.com/chainer/chainer

@@ -37,7 +37,7 @@ landing_page:
<a target="_blank" class="external" href="https://colab.research.google.com/github/tensorflow/docs/blob/master/site/en/r2/tutorials/quickstart/beginner.ipynb">пример “Hello World”</a>,
затем прочитайте руководства и учебники чтобы узнать больше.</p>
<h3>Для экпертов</h3>
The Subclassing API представляет define-by-run интерфейс для продвинутых исследований.
Subclassing API представляет define-by-run интерфейс для продвинутых исследований.
Создайте класс для своей модели, затем напишите forward pass
императивно. С легкостью программируйте кастомные слои, функции активации и циклы обучения.<br>

This comment has been minimized.

Copy link
@0101011

0101011 Jun 18, 2019

Contributor

Мы раньше переводили "кастомные" как "собственные" для чистоты документации, но можно оставить и так в качестве эксперимента - суть понятна :)

This comment has been minimized.

Copy link
@stabuev

stabuev Jun 19, 2019

Author Contributor

собственные мне тоже больше нравится :)

@@ -0,0 +1,306 @@
# Эффективный TensorFlow 2.0

This comment has been minimized.

Copy link
@0101011

0101011 Jun 18, 2019

Contributor

До бета версии переводили старую версию этого документа, крупные блоки текста остались без изменений (https://github.com/tensorflow/docs/blob/master/site/ru/guide/effective_tf2.md)

Я смогу сравнить изменения в течение 2 дней (вечер по московскому времени)

@@ -0,0 +1,72 @@
# Обновите код до TensorFlow 2.0

TensorFlow 2.0 включает много изменений API, таких как изменение порядка аргументов, переименование символов и изменение значений по умолчанию параметров. Выполнение этих модификаций вручную утомительно и подвержено ошибкам. Для упрощения изменений и как можно более плавного перехода на TF 2.0, команда TensorFlow создала утилиту `tf_upgrade_v2`, помогающую перейти от устаревшего кода к новому API.

This comment has been minimized.

Copy link
@0101011

0101011 Jun 18, 2019

Contributor

"Выполнение этих модификаций вручную":

Лишний пробел после "выполнение"


Утилита `tf_upgrade_v2` добавляется автоматически с `pip install` TF 2.0. Она ускорит процесс обновления за счет преобразования существующих скриптов TensorFlow 1.x Python в TensorFlow 2.0.

Скрипт обновленияч автоматизирует максимум возможного, но все еще есть синтаксические и стилистические изменения, которые не могут быть выполнены скриптом.

This comment has been minimized.

Copy link
@0101011

0101011 Jun 18, 2019

Contributor

"Скрипт обновления"

# обновите файлы .py и скопируйте все остальные файлы в outtree
tf_upgrade_v2 --intree coolcode --outtree coolcode-upgraded
# всего лишь обновление .py файлов

This comment has been minimized.

Copy link
@0101011

0101011 Jun 18, 2019

Contributor

обновление только .py файлов

New: tf.argmax(input=[[1, 3, 2]], axis=0))
```
Вся эта информация добавляется в файл`report.txt` который будет экспортирован в вашу текущую директорию. После выполнения `tf_upgrade_v2` и экспорта вашего обновленного скрипта, вы можете запустить модель и убедиться что результат аналогичен TF 1.x.

This comment has been minimized.

Copy link
@0101011

0101011 Jun 18, 2019

Contributor
  1. Запятая после в файлreport.txt
  2. Директорию => папку
  3. Запятая после "модель и убедиться"

## Предостережения

- Не обновляйте части вашего кода вручную до запуска этого скрипта. В частности, функции поменявшие порядок аргументов, такие как `tf.argmax` или `tf.batch_to_space` вынудят скрипт неправильно добавить имена аргументов, что приведет к ошибкам в существующем коде.

This comment has been minimized.

Copy link
@0101011

0101011 Jun 18, 2019

Contributor
  1. Запятая после "функции"

- Не обновляйте части вашего кода вручную до запуска этого скрипта. В частности, функции поменявшие порядок аргументов, такие как `tf.argmax` или `tf.batch_to_space` вынудят скрипт неправильно добавить имена аргументов, что приведет к ошибкам в существующем коде.

- Скрипт предполагает что `tensorflow` импотирован с использованием `import tensorflow as tf`.

This comment has been minimized.

Copy link
@0101011

0101011 Jun 18, 2019

Contributor

Импортирован :)


- Скрипт предполагает что `tensorflow` импотирован с использованием `import tensorflow as tf`.

- Этот скрипт не меняет аргументы. Вместо этого скрипт добавляет ключи аргументов к функциям у которых изменися порядок аргументов.

This comment has been minimized.

Copy link
@0101011

0101011 Jun 18, 2019

Contributor

Запятая после "к функциям"


- Этот скрипт не меняет аргументы. Вместо этого скрипт добавляет ключи аргументов к функциям у которых изменися порядок аргументов.

- Проверьте [tf2up.ml] (http://tf2up.ml) для удобного инструментаобновления Jupyter

This comment has been minimized.

Copy link
@0101011

0101011 Jun 18, 2019

Contributor

Проверьте [tf2up.ml] (http://tf2up.ml) для удобного инструмента обновления файлов в форматах Jupyter Notebook и Python в репозитории GitHub.

@0101011

This comment has been minimized.

Copy link
Contributor

commented Jun 18, 2019

  1. That's it for the typos and a couple of wording improvements.

  2. We should also compare and contrast the pre-beta Effective TF 2.0 doc for changes (https://github.com/tensorflow/docs/blob/master/site/ru/guide/effective_tf2.md):

Would appreciate any help or head-ups with this doc.

  1. All in all great translations @stabuev, clear and to the point - I'd definitely read some more from you!
@stabuev

This comment has been minimized.

Copy link
Contributor Author

commented Jun 18, 2019

thanks for review, I'll correct all typos and add improvements tomorrow.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.