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

Resource Citation #83

Merged
merged 6 commits into from
Aug 30, 2023
Merged

Conversation

20001LastOrder
Copy link
Collaborator

Use better logic to handle resource citations. Also, in this PR:

  1. Change the Markdown to Slack post-processor into a post-processor class
  2. Fix a problem in the Google Search tool.

@20001LastOrder
Copy link
Collaborator Author

@YujingYang666777 Please take a look

saminegash
saminegash previously approved these changes Aug 16, 2023
@saminegash
Copy link
Contributor

rebase please

@saminegash
Copy link
Contributor

rebase this as well please

@20001LastOrder
Copy link
Collaborator Author

@saminegash I resolve the merge conflict

@amirfz
Copy link
Contributor

amirfz commented Aug 23, 2023

@20001LastOrder the message says that there's still conflict in task agent and tools

@saminegash
Copy link
Contributor

And also can you rebase on top of thw origin/main branch as @oshoma suggestion on the doc

@20001LastOrder
Copy link
Collaborator Author

@saminegash Rebased

Copy link
Contributor

@saminegash saminegash left a comment

Choose a reason for hiding this comment

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

Lgtm

@saminegash saminegash merged commit 54a54b6 into Aggregate-Intellect:main Aug 30, 2023
@20001LastOrder 20001LastOrder deleted the 'url-fixing' branch September 14, 2023 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants