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.

Sign up for GitHub

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

Jump to bottom

Prelaunch #41

Merged
merged 9 commits into from
Aug 5, 2024
Merged

Prelaunch #41

merged 9 commits into from
Aug 5, 2024

Conversation

CallmeNafiy
Copy link
Collaborator

  • Removed op.parse() and op.instruct() and related notebooks and bash script
  • Added README for poppler installation
  • Added logo image for aesthetic
  • Tested individual notebooks on file

@CallmeNafiy CallmeNafiy requested a review from Sdddell as a code owner August 3, 2024 01:10
@CallmeNafiy CallmeNafiy requested a review from jojortz August 3, 2024 01:10
Sdddell
Sdddell approved these changes Aug 3, 2024
View reviewed changes
Copy link
Collaborator

@Sdddell Sdddell left a comment

Choose a reason for hiding this comment

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

It seems the test_example.py did not pass the format check.

CambioML
CambioML suggested changes Aug 5, 2024
View reviewed changes
Copy link
Collaborator

@CambioML CambioML left a comment

Choose a reason for hiding this comment

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

Please address the comments to add note to README

README.md Outdated
@@ -4,7 +4,7 @@ AnyParser provides an API to accurately extract your unstructured data (e.g. PDF

## :seedling: Set up your AnyParser API key

AnyParser is still in private beta. If you are interested in testing our document models, please reach out at [email protected] for a FREE testing API key.
You can generate your keys at the [Playground Account Page](https://www.cambioml.com/account) with up to 2 keys and 100 total free pages per account. If you're interested in more AnyParser usage and applications, please reach out at [email protected] for details.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please add more details on the 10 page limit per call

@CambioML CambioML merged commit 3236c3e into main Aug 5, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CambioML CambioML CambioML approved these changes

@Sdddell Sdddell Sdddell approved these changes

@jojortz jojortz Awaiting requested review from jojortz

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@CallmeNafiy @Sdddell @CambioML @jojortz