- Notifications
You must be signed in to change notification settings - Fork5k
docs(taosx): kinghistorian doc #TD-38665#33758
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Conversation
Summary of ChangesHello@zyyang90, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request delivers new documentation outlining the process of integrating KingHistorian with TDengine TSDB. It provides users with clear instructions on how to set up and manage data migration and real-time synchronization tasks via the Explorer interface, facilitating efficient data transfer and management. Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on ourdocumentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on@gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign uphere. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with theGemini Code Assist IDE Extension. Footnotes
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Code Review
This pull request adds documentation for KingHistorian data integration. The changes are a good starting point, but there are several areas for improvement. I've identified a redundant and incorrect Chinese documentation file that should be removed. Additionally, both the English and Chinese versions have minor grammatical issues, including run-on sentences and inconsistent punctuation, which affect readability. Some instructions are also a bit vague and could be clarified to be more helpful to the user. My review includes specific suggestions to address these points and improve the overall quality of the documentation.
Uh oh!
There was an error while loading.Please reload this page.
| ###2. Configure Basic Information | ||
| Enter the task name in**Name**, e.g., "test_kinghistorian"; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
The semicolon at the end of this line is grammatically incorrect and inconsistent with the rest of the document. Please replace it with a period for correctness and consistency.
| Enter the task name in**Name**, e.g., "test_kinghistorian"; | |
| Enter the task name in**Name**, e.g., "test_kinghistorian". |
| ###4. Configure Tags | ||
| In the**Upload CSV Configuration File**, click**Download Data Points**, select the required**Point Group**,**Points**, and**Tags**. A CSV configuration file will be generated and downloaded locally. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
This line contains a run-on sentence (comma splice), which can be confusing for readers. It should be rephrased for grammatical correctness and clarity.
| In the**Upload CSV Configuration File**, click**Download Data Points**, select the required**Point Group**,**Points**, and**Tags**.ACSV configuration filewill be generatedanddownloaded locally. | |
| In the**Upload CSV Configuration File**, click**Download Data Points**,thenselect the required**Point Group**,**Points**, and**Tags**.This action generates aCSV configuration file anddownloads it locally. |
| ###6. Configure Advanced Options | ||
| Fill in the relevant configuration parameters in the**Advanced Options** area. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
This instruction is vague. To make the documentation more helpful, consider briefly mentioning what kind of parameters can be configured in 'Advanced Options', or link to more detailed documentation if available. If there are no specific parameters to highlight, a more general phrasing would be better.
| Fill in the relevant configuration parametersin the**Advanced Options** area. | |
| Configure any additional settingsin the**Advanced Options** area as needed. |
Uh oh!
There was an error while loading.Please reload this page.
| **代理** 是非必填项,如果需要,可以在下拉框中选择指定的代理,也可以先点击右侧的 **+创建新的代理** 按钮 | ||
| 在 **目标数据库** 下拉列表中选择一个目标数据库,也可以先点击右侧的 **+创建数据库** 按钮 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
| ### 4. 配置 Tag | ||
| 在 **上传 CSV 配置文件**中,点击 **下载数据点位**,选择需要 **测点组**,**测点**,**标签**,之后会生成 CSV 配置文件,并下载到本地。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
This line is a run-on sentence (comma splice) and uses standard commas for a list of items where enumeration commas (、) are more appropriate in Chinese. For better readability and grammatical correctness, please rephrase the sentence and use the correct punctuation.
在 **上传 CSV 配置文件**中,点击 **下载数据点位**,选择需要的**测点组**、**测点**和**标签**,这会生成一个 CSV 配置文件并下载到本地。| ### 6. 配置高级选项 | ||
| 在 **高级选项** 区域填写高级选项相关的配置参数。 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
This instruction is vague. To make the documentation more helpful, consider briefly mentioning what kind of parameters can be configured in 'Advanced Options', or link to more detailed documentation if available. If there are no specific parameters to highlight, a more general phrasing would be better.
在 **高级选项** 区域根据需要配置其他相关参数。
zitsen left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Please fix the comments
Description
docs(taosx): kinghistorian doc #TD-38665
Jira:https://jira.taosdata.com:18080/browse/TD-38665
Checklist
Please check the items in the checklist if applicable.