Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

feat: add resume_extract to sdk #57

Closed
wants to merge 6 commits into from
Closed

feat: add resume_extract to sdk #57

wants to merge 6 commits into from

Conversation

CambioML
Copy link
Collaborator

@CambioML CambioML commented Oct 25, 2024

Feature

Add resume_extract to sdk. Also add example notebook pdf_resume_to_json.ipynb.

Use

resume_extract_result, time_elapsed = ap.resume_extract(file_path, extract_type)

  • file_path: path to your local file
  • extract_type: one of 'pii', 'education', 'work_experience', 'personal_info', 'skills', 'certifications', or 'projects'

Testing

  • ran all automated tests in tests/test.py
    • includes new test_sync_extract_resume.py

TODO

@CambioML CambioML requested a review from Sdddell as a code owner October 25, 2024 20:20
@@ -187,6 +189,74 @@ def extract_key_value(
else:
return f"Error: {response.status_code} {response.text}", None

def resume_extract(
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

nit: are we planning to add async as well?

Copy link
Contributor

@jojortz jojortz Nov 2, 2024

Choose a reason for hiding this comment

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

FEATURE

  • Added async resume extract
  • Renamed to extract_resume_key_value

TESTING

  • added unittest for async resume extract
  • Tested sync and async resume extract on preprod-api

NOTE/TODO
Testing will fail since async_resume is not in public yet. However, once that is pushed to public, the extract key value tests will fail.

@lingjiekong
Copy link
Member

close due to duplication in #60

@lingjiekong lingjiekong closed this Nov 5, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants