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

show more info about datasets #464

Merged
merged 3 commits into from
Mar 14, 2024
Merged

show more info about datasets #464

merged 3 commits into from
Mar 14, 2024

Conversation

amulil
Copy link
Contributor

@amulil amulil commented Mar 11, 2024

No description provided.

@pppppM pppppM requested a review from HIT-cwh March 11, 2024 12:29
@HIT-cwh
Copy link
Collaborator

HIT-cwh commented Mar 12, 2024

Thank you for your contribution; it is greatly appreciated. For enhanced clarity, may I suggest printing labels that are equal to -100 with a distinctive character, such as [-100]? This would help to distinguish them more effectively.

@amulil
Copy link
Contributor Author

amulil commented Mar 14, 2024

Thank you for your contribution; it is greatly appreciated. For enhanced clarity, may I suggest printing labels that are equal to -100 with a distinctive character, such as [-100]? This would help to distinguish them more effectively.

@HIT-cwh Thanks for your advice! I've changed it.

@HIT-cwh HIT-cwh merged commit 164d383 into InternLM:main Mar 14, 2024
1 check passed
@amulil amulil deleted the patch-1 branch March 18, 2024 12:43
llkn-2 pushed a commit to llkn-2/xtuner that referenced this pull request Jul 31, 2024
* show more info about datasets

* update log_dataset.py

* update log_dataset.py
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.

2 participants