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.

Sign up for GitHub

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

Jump to bottom

数据集文档更新 #140

Merged
merged 5 commits into from
Nov 21, 2024
Merged

数据集文档更新 #140

merged 5 commits into from
Nov 21, 2024

Conversation

Songjoy
Copy link
Contributor

@Songjoy Songjoy commented Oct 16, 2024 •
edited by windsonsea
Loading

Drun 版本更新至 v2.3.0，模型微调功能调整较大，故对此的文档进行更新修改

See preview:

https://deploy-preview-140--unrivaled-kringle-809f36.netlify.app/dtx/dataset

@github-actions github-actions bot added dtx size/XL A PR that changes 500-999 lines labels Oct 16, 2024
@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@netlify Netlify
Copy link

netlify bot commented Oct 16, 2024 •
edited
Loading

✅ Deploy Preview for unrivaled-kringle-809f36 ready!

Name Link
🔨 Latest commit 8b3a633
🔍 Latest deploy log https://app.netlify.com/sites/unrivaled-kringle-809f36/deploys/673e91b28874c80008c2c0e4
😎 Deploy Preview https://deploy-preview-140--unrivaled-kringle-809f36.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

windsonsea
windsonsea reviewed Oct 17, 2024
View reviewed changes
Copy link
Collaborator

@windsonsea windsonsea left a comment •
edited
Loading

Choose a reason for hiding this comment

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

这种大量的修改，不说明一下原因吗？

image

新增或删除文件，需要体现在：docs/zh/navigation.yml

docs/zh/docs/dtx/index.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/index.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/index.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/index.md Show resolved Hide resolved
docs/zh/docs/dtx/index.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/index.md Show resolved Hide resolved
docs/zh/docs/dtx/parameters.md Outdated Show resolved Hide resolved
@windsonsea windsonsea added the wip-do-not-merge work in process or a draft label Oct 17, 2024
@windsonsea windsonsea marked this pull request as draft October 17, 2024 03:09
@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

windsonsea
windsonsea reviewed Oct 18, 2024
View reviewed changes
docs/zh/docs/dtx/dataset.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/dataset.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/dataset.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/dataset.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/dataset.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/dataset.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/dataset.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/fine-tuning-lab.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/fine-tuning-lab.md Outdated Show resolved Hide resolved
docs/zh/docs/dtx/fine-tuning-fast.md Outdated Show resolved Hide resolved
@windsonsea
Copy link
Collaborator

Hi @Songjoy , 你把缺少的文件补齐，消除 404 问题，格式我来改吧

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@Songjoy Songjoy closed this Nov 20, 2024
@Songjoy Songjoy reopened this Nov 20, 2024
@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@Songjoy
Copy link
Contributor Author

Songjoy commented Nov 20, 2024

Hi @Songjoy , 你把缺少的文件补齐，消除 404 问题，格式我来改吧

非常感谢🙏您，缺少文件已经补齐，格式我这边已经调整。

@windsonsea
Copy link
Collaborator

收到，今天休假了，明天来处理，谢谢😊

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@github-actions GitHub Actions
Copy link

This PR exceeds the recommended size of 1000 lines. Please make sure you are NOT addressing multiple issues with one PR. Note this PR might be rejected due to its size.

@windsonsea windsonsea marked this pull request as ready for review November 21, 2024 01:52
@windsonsea windsonsea merged commit 17c4edb into d-run:main Nov 21, 2024
6 checks passed
@windsonsea windsonsea removed the wip-do-not-merge work in process or a draft label Nov 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@windsonsea windsonsea windsonsea approved these changes

Assignees
No one assigned
Labels
size/XL A PR that changes 500-999 lines
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Songjoy @windsonsea