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

feat:add library to data process #30

Merged
merged 1 commit into from
Mar 8, 2024
Merged

Conversation

wangxinbiao
Copy link
Collaborator

@wangxinbiao wangxinbiao commented Mar 4, 2024

What type of PR is this?

What this PR does / why we need it

add library to data process
pdf transform csv
add cli convert pdf

Which issue(s) this PR fixes

#29
#15

Special notes for your reviewer

@github-actions github-actions bot requested a review from bjwswang March 4, 2024 07:37
Copy link
Collaborator

@bjwswang bjwswang left a comment

Choose a reason for hiding this comment

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

What is mapper ?

@wangxinbiao
Copy link
Collaborator Author

What is mapper ?

Edit and transform the data.

@wangxinbiao wangxinbiao force-pushed the main branch 7 times, most recently from aeb8627 to bbdc7bc Compare March 7, 2024 06:25
self._chunk_size = chunk_size
self._chunk_overlap = chunk_overlap

def transformer(self):
Copy link
Collaborator

Choose a reason for hiding this comment

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

transformer -> transform

@wangxinbiao wangxinbiao force-pushed the main branch 6 times, most recently from a614649 to 95728b1 Compare March 7, 2024 09:18
@wangxinbiao wangxinbiao force-pushed the main branch 2 times, most recently from 789cb01 to 5ae0c61 Compare March 8, 2024 02:51
@bjwswang
Copy link
Collaborator

bjwswang commented Mar 8, 2024

@wangxinbiao pls resolve the conflicts

@wangxinbiao
Copy link
Collaborator Author

@wangxinbiao pls resolve the conflicts

resolved

Copy link
Collaborator

@bjwswang bjwswang left a comment

Choose a reason for hiding this comment

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

Thanks!

@bjwswang bjwswang merged commit 517a206 into kubeagi:main Mar 8, 2024
3 checks 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