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

docs: Polish ReadME #1635

Merged
merged 16 commits into from
Mar 2, 2025
Merged

docs: Polish ReadME #1635

merged 16 commits into from
Mar 2, 2025

Conversation

Douglasymlai
Copy link
Collaborator

@Douglasymlai Douglasymlai commented Feb 21, 2025

Description

This polishes the ReadMe content, graphics and layout. close #1256

Checklist

Go over all the following points, and put an x in all the boxes that apply.

  • I have read the CONTRIBUTION guide (required)
  • I have linked this PR to an issue using the Development section on the right sidebar or by adding Fixes #issue-number in the PR description (required)
  • I have checked if any dependencies need to be added or updated in pyproject.toml and poetry.lock
  • I have updated the tests accordingly (required for a bug fix or a new feature)
  • I have updated the documentation if needed:
  • I have added examples if this is a new feature

If you are unsure about any of these, don't hesitate to ask. We are here to help!

@Douglasymlai Douglasymlai added the documentation Improvements or additions to documentation label Feb 21, 2025
@Douglasymlai Douglasymlai self-assigned this Feb 21, 2025
@Douglasymlai Douglasymlai linked an issue Feb 21, 2025 that may be closed by this pull request
2 tasks
@Douglasymlai Douglasymlai added P0 Task with high level priority enhancement New feature or request labels Feb 21, 2025
Copy link
Member

@Wendong-Fan Wendong-Fan left a comment

Choose a reason for hiding this comment

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

thanks @Douglasymlai , left some comments below

Copy link
Member

Choose a reason for hiding this comment

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

the last sentence of the doc in the image should be removed

Comment on lines +243 to +251
> **Data (Hosted on Hugging Face)**

| Dataset | Chat format | Instruction format | Chat format (translated) |
|----------------|-----------------------------------------------------------------------------------------------------|------------------------------------------------------------------------------------------------------------------|--------------------------------------------------------------------------------------------|
| **AI Society** | [Chat format](https://huggingface.co/datasets/camel-ai/ai_society/blob/main/ai_society_chat.tar.gz) | [Instruction format](https://huggingface.co/datasets/camel-ai/ai_society/blob/main/ai_society_instructions.json) | [Chat format (translated)](https://huggingface.co/datasets/camel-ai/ai_society_translated) |
| **Code** | [Chat format](https://huggingface.co/datasets/camel-ai/code/blob/main/code_chat.tar.gz) | [Instruction format](https://huggingface.co/datasets/camel-ai/code/blob/main/code_instructions.json) | x |
| **Math** | [Chat format](https://huggingface.co/datasets/camel-ai/math) | x | x |
| **Physics** | [Chat format](https://huggingface.co/datasets/camel-ai/physics) | x | x |
| **Chemistry** | [Chat format](https://huggingface.co/datasets/camel-ai/chemistry) | x | x |
Copy link
Member

Choose a reason for hiding this comment

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

this part shouldn't under 2. Utilize Various LLMs as Backends

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Should we remove this part entirely, or should we place it somewhere else?

README.md Outdated
| **Chemistry** | [Chat format](https://huggingface.co/datasets/camel-ai/chemistry) | x | x |
| **Biology** | [Chat format](https://huggingface.co/datasets/camel-ai/biology) | x | x |

### 3. Visualizations of Instructions and Tasks
Copy link
Member

Choose a reason for hiding this comment

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

this part shouldn't belong to Tech Stack, same as 2. Utilize Various LLMs as Backends

README.md Outdated
| **Code** | [Instructions](https://atlas.nomic.ai/map/902d6ccb-0bbb-4294-83a8-1c7d2dae03c8/ace2e146-e49f-41db-a1f4-25a2c4be2457) | [Tasks](https://atlas.nomic.ai/map/efc38617-9180-490a-8630-43a05b35d22d/2576addf-a133-45d5-89a9-6b067b6652dd) |
| **Misalignment** | [Instructions](https://atlas.nomic.ai/map/5c491035-a26e-4a05-9593-82ffb2c3ab40/2bd98896-894e-4807-9ed8-a203ccb14d5e) | [Tasks](https://atlas.nomic.ai/map/abc357dd-9c04-4913-9541-63e259d7ac1f/825139a4-af66-427c-9d0e-f36b5492ab3f) |

<br>

## Installation
Copy link
Member

Choose a reason for hiding this comment

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

how about remove the detailed Installation guidance in readme and set one separated file for Installation? just like how we did for contribution guideline https://github.com/camel-ai/camel/blob/master/CONTRIBUTING.md

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yes, that sounds like a good plan. I believe the best approach would be to include the detailed installation in the documentation and maintain a brief summary in the repository's ReadMe file.

README.md Outdated

Please reachout to us on [CAMEL discord](https://discord.camel-ai.org/) if you encounter any issue set up CAMEL.

> If you would like to contribute to CAMEL refer to our [Contributing Guidelines](https://github.com/camel-ai/camel/blob/master/CONTRIBUTING.md).
Copy link
Member

Choose a reason for hiding this comment

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

this shouldn't under ### Seeking Help

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I designed this section to encourage the conversion of individuals joining our community. Are there any other channels through which we can support our users?

README.md Outdated

- `Self-Instruct` from *Yizhong Wang et al.*: [SELF-INSTRUCT: Aligning Language Models with Self-Generated Instructions](https://arxiv.org/pdf/2212.10560). [[Example](https://github.com/camel-ai/camel/blob/master/examples/datagen/self_instruct/self_instruct.py)]
## Research
Copy link
Member

Choose a reason for hiding this comment

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

for this part could we move it up, after Tech Stack session?

README.md Outdated
>We also welcome you to help CAMEL grow by sharing it on social media, at events, or during conferences. Your support makes a big difference!


## Community & Contact
Copy link
Member

Choose a reason for hiding this comment

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

this part should be moved up, let's adjust the order based on importance,Citation and License could in the bottom

@Wendong-Fan Wendong-Fan changed the title [1256 feature request] Polish ReadME docs: Polish ReadME Feb 22, 2025
@Wendong-Fan Wendong-Fan added this to the Sprint 23 milestone Feb 22, 2025
@Douglasymlai
Copy link
Collaborator Author

I have updated the PR based on the comments. @Wendong-Fan

Copy link
Member

@Wendong-Fan Wendong-Fan left a comment

Choose a reason for hiding this comment

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

Thanks @Douglasymlai !

@Wendong-Fan Wendong-Fan merged commit 42977bc into master Mar 2, 2025
5 of 6 checks passed
@Wendong-Fan Wendong-Fan deleted the 1256-feature-request-polish-readme branch March 2, 2025 10:51
ZackZikaiXiao pushed a commit to ZackZikaiXiao/camel that referenced this pull request Mar 23, 2025
Co-authored-by: Wendong <w3ndong.fan@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request P0 Task with high level priority
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

[Feature Request] Polish README
2 participants