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] Added READMEs in each directory #90

Merged
merged 7 commits into from
Aug 4, 2024

Conversation

om-raheja
Copy link
Collaborator

This is my documentation with README.md files in each directory slightly modified. I believe running "git rebase" didn't work for me again, even though locally it appears everything is squashed. I request the maintainers to make adjustments as necessary, although I believe what I made in this hour should hold value for anyone who wants to contribute to OpenAGI itself and to improve it's further maintenance. Thanks.

Copy link
Collaborator

@dongyuanjushi dongyuanjushi left a comment

Choose a reason for hiding this comment

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

Could you pull the latest files in the openagi and remove the deprecated files in the openagi/src and use the latest version in the pyopenagi? This is because openagi has been recently refactored, so some of the files you have uploaded may not work. Thank you!

@om-raheja
Copy link
Collaborator Author

Yeah, I had to deal with that when merging. I removed the openagi/ directory and it should work okay now.

setup.py Outdated Show resolved Hide resolved
Copy link
Collaborator

@dongyuanjushi dongyuanjushi left a comment

Choose a reason for hiding this comment

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

Could you do the changes for some specific files/folders that are mentioned above?

Copy link
Collaborator

Choose a reason for hiding this comment

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

The title of this file remains to be changed.

Copy link
Collaborator

@dongyuanjushi dongyuanjushi left a comment

Choose a reason for hiding this comment

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

Could you do the changes mentioned above?

@dongyuanjushi dongyuanjushi merged commit 6ca6c94 into agiresearch:main Aug 4, 2024
1 check passed
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