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

The Chinese translation is complete. Request a review. #307

Open
wants to merge 623 commits into
base: main
Choose a base branch
from

Conversation

huzhenbang
Copy link

The Chinese translation is complete. Request a review.

Alexis Ronez and others added 30 commits June 17, 2019 18:17
Because nothing is more important than an introduction to help the reader dive into something as hard as Vulkan
SaschaWillems and others added 13 commits July 3, 2022 14:03
)

* Prefer use of dynamic state for viewport and scissor

* Slight rewording, working in-page anchor

* Updated command buffers chapter and code to use dynamic viewport and scissor state

* Updated swap chain recreation chapter

No more need to recreate the graphics pipeline as viewport and scissor state are now dynamic

* Updated command buffer chapter code to use dynamic viewport and scissor

* Removed render pass destruction/creation for swap chain recreation

This is not needed anymore

* Use dynamic state for viewport and scissor for samples after the fixed function chapter

* Use dynamic state for viewport and scissor for samples after the swap chain recreation chapter

Removed render pass destruction/creation for swap chain recreation, no longer necessary

* Removed duplicate code

* Wording change

* Changed wording on chapter intro, moved dynamic state paragraph to the top

* Fixed wording

* Fixed wording

* C++ style cast

* Wording

* Wording

* Wording

* Added note on renderpass recreation

* Apply suggestions from code review

Incorporate suggestions

Co-authored-by: Alexander Overvoorde <overv161@gmail.com>

* Added dynamic state to conclusion chapter

* Remove unnecessary link

* Remove code referencing command buffers too early

Co-authored-by: Alexander Overvoorde <overv161@gmail.com>
append new section showing how to fix `VK_ERROR_INCOMPATIBLE_DRIVER` with latest MoltenVK sdk
Fix the recreateSwapChain() function to match the code, which may cause the "Validation Error" if you resize and close window.
On Fedora, the packages should be libXi-devel and libXxf86vm-devel,
instead of libXi and libXxf86vm. Otherwise, by following the tutorial,
we might get errors such as:

/usr/bin/ld: cannot find -lXxf86vm
collect2: error: ld returned 1 exit status
make: *** [Makefile:5: VulkanTest] Error 1
@Overv
Copy link
Owner

Overv commented Nov 3, 2022

Any native speakers who can pitch in to review this? @chirsz-ever @fangcun010

@chirsz-ever
Copy link
Contributor

翻译应该放在 zh 文件夹并中并在 config.json 中加入中文配置,参见 chirsz-ever@6802acb

@chirsz-ever
Copy link
Contributor

build_ebook.py 换行符变成了 CRLF,应该维持原样使用 LF。

如果是在 Windows 上进行开发可以将 git 配置为提交时使用 LF,checkout 到文件系统中时使用 CRLF1

git config --global core.autocrlf true

配置后建议使用 git add --renormalize . 清理工作区。

Footnotes

  1. https://git-scm.com/book/zh/v2/%E8%87%AA%E5%AE%9A%E4%B9%89-Git-%E9%85%8D%E7%BD%AE-Git#_%E6%A0%BC%E5%BC%8F%E5%8C%96%E4%B8%8E%E5%A4%9A%E4%BD%99%E7%9A%84%E7%A9%BA%E7%99%BD%E5%AD%97%E7%AC%A6

@chirsz-ever
Copy link
Contributor

chirsz-ever commented Nov 5, 2022

duax.io 处理中文 URL 有问题,文件名应当改用英文,并在文件开头使用元数据块来指明中文标题

中文路径问题将在 Overv/daux.io#2 被合入后解决。

@chirsz-ever
Copy link
Contributor

发现的一些问题:

  • 不少地方原文用行内代码块(`xxx`)的翻译过来后改成双引号了;
  • 半角和全角标点混用以及错用,有时导致 Markdown 语法错误;
    此第三方服务的完整隐私政策可查阅 [https://help.disqus.com/terms-and-policies/disqus-privacy-policy](https://help.disqus.com/terms-and-policies/disqus-privacy-policy)。
  • 没有注意汉字与英文单词、行内代码块之间的空格;
    该对象将随着Vulkan实例VkInstance销毁而自动销毁,所以我们不必在`cleanup`函数中添加额外的处理。
  • Mipmap 应翻译为“多级渐远纹理”

排版相关可以参考 中文排版指北

图形学术语可以参考《游戏引擎架构》中英词汇索引表

参考 这里 的建议,某些术语可以保留英文原文,如使用“uniform buffer”或“uniform 缓冲”而非“统一缓冲区”,“统一的缓冲区”。

@orbitgw
Copy link

orbitgw commented Apr 19, 2023

When will the translation be deployed? @Overv

@Overv
Copy link
Owner

Overv commented Apr 19, 2023

@orbitgw Would you be willing to go through the chapters and review them? If a native speaker can confirm that it looks good then this can be deployed.

@orbitgw
Copy link

orbitgw commented Apr 20, 2023

@Overv
I would be happy to do it, but it will take time. Cause I am currently a high school student and am about to face the college entrance examination. I need to put more energy into my studies. I will graduate in June 2024, and by then, I will have plenty of time to do this work.

@Overv
Copy link
Owner

Overv commented Apr 20, 2023

@orbitgw That is fine. Perhaps someone else has a chance to look at it earlier.

@chirsz-ever
Copy link
Contributor

This PR is a mess with unrelatied modifications and wrong format. The author did not take action or reply to comments for half a year. I think I could start a new translation PR in some days.

@Overv How do you think about Overv/daux.io#2? If you think it change too much, It's OK to just add Overv/daux.io@b3ab5c3 to remove hard-coded "Introduction" path, and I will use English file names in the Chinese translation.

@orbitgw
Copy link

orbitgw commented Apr 21, 2023

@chirsz-ever

This PR is a mess with unrelatied modifications and wrong format. The author did not take action or reply to comments for half a year. I think I could start a new translation PR in some days.

@Overv How do you think about Overv/daux.io#2? If you think it change too much, It's OK to just add Overv/daux.io@b3ab5c3 to remove hard-coded "Introduction" path, and I will use English file names in the Chinese translation.

我赞同你的想法,用中文做路径太奇怪了,而且不好访问

@orbitgw
Copy link

orbitgw commented Apr 21, 2023

@Overv
I have roughly browsed through it and there is a suspicion that this PR may have completely used translation software without manual verification.

@orbitgw
Copy link

orbitgw commented Apr 21, 2023

@chirsz-ever 我想咱们可以合作一下

@Overv
Copy link
Owner

Overv commented Apr 24, 2023

我同意,用英文保留文件名会更方便,这样它们就更容易与其他版本匹配了

翻译准备好发布时,我会修复介绍路径

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.

None yet