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

Improve use of async DocParse via aryn-sdk #1128

Closed
wants to merge 68 commits into from
Closed

Improve use of async DocParse via aryn-sdk #1128

wants to merge 68 commits into from

Conversation

MarkLindblad
Copy link
Contributor

No description provided.

@MarkLindblad
Add partition_file_submit_async
d6f8cb4
@MarkLindblad
Add partition_file_result_async
2216c01
@MarkLindblad
Fix linting
eb327a8
@MarkLindblad
Add docstring, return type for partition_file_submit_async
fec3c51
@MarkLindblad
Fix linting
dc8059f
@MarkLindblad
Make partition_file_result_async easier to use
735c828
@MarkLindblad
Add partition_file_result_async docstring
f2ebd14
@MarkLindblad
Fix linting
cc0ce7c
@MarkLindblad
Add tests
c80bbdd
@MarkLindblad
Add general error when unexpected http code encountered
85678fb
@MarkLindblad
Fix linting
a220bef
@MarkLindblad
Rename NoSuchAsyncPartitionerJob to NoSuchAsyncPartitionerJobError
784122e
@MarkLindblad
Fix test_partition with current exception (PartitionError not Value…
e08bd74 
…Error)
@MarkLindblad
Fix expected type of Exception in test_partition_it_zero_page
6c4b9dd
@MarkLindblad
Update reference json for test_partition_it integ test
3d73b60
@MarkLindblad
Add support for positional arguments in partition_file_submit_async
2ed99a2
@MarkLindblad
Add explanatory comments
aae7356
@MarkLindblad
Add support for positional arguments in partition_file_submit_async
bcd92af
@MarkLindblad
Fix handling of positional and keyword arguments in `partition_file_s…
3e2c970 
…ubmit_async`
@MarkLindblad
Fix linting
c68cda6
@MarkLindblad
Move async tests together
3d2eac3
@MarkLindblad
Add url forwarding validation
e5d2f88
@MarkLindblad
Add test_partition_file_submit_async unit test
d4b072c
@MarkLindblad
Fix mypy issues
e55687e
@MarkLindblad
Fix relevant docstrings. Add a multi-job example
a82b90c
@MarkLindblad
Add async examples to readme
c38418d
@MarkLindblad
Narrow excepted Exceptions in multi-document async examples
d081f9c
@MarkLindblad
Add synchronous test checking unsupported file format behavior
cd94238
@MarkLindblad
Make partition_file_submit_async response more intuitive
ab6aa3f
@MarkLindblad
Fix examples in README
49a393b
@MarkLindblad
Rename _partition_file_inner's _webhook_dest_url to just `webhook…
b7be7a8 
…_url`
@MarkLindblad
Fix function names in README, change double-call loop to single-call …
e16f926 
…loop in multi-async call example
@MarkLindblad
Make return type of partition_file_async_submit more strict
e572033
@MarkLindblad
Remove pydantic, enum
4fd7721
@MarkLindblad
Remove use of inspect, add * barrier to parameters, parse urls
14c146e
@MarkLindblad
Change two-call loop to single-call loop in `test_partition_file_asyn…
4df23ce 
…c_with_unsupported_file_format`
@MarkLindblad
Change two-call loop to single-call loop in `test_multiple_partition_…
75646ba 
…file_async`
@MarkLindblad
Fix and improve async aryn-sdk examples
e2edada
@MarkLindblad
Treat all 2** status codes as successful
a0ae467
@MarkLindblad
Simplify mocked function sigature
cd31818
@MarkLindblad
Add documentation for orientation correction, add test
61a21af
@MarkLindblad
Add partition_file_async_list to aryn-sdk, multi-test
419a63b
@MarkLindblad
Make return type of partition_file_async_result more strict
b9abcd0
@MarkLindblad
Make return type of partition_file_async_list more strict
0ae3757
@MarkLindblad
Copy link
Contributor Author

Now adding changes based on discussion after #1116 was merged.

@MarkLindblad MarkLindblad reopened this Jan 27, 2025
@MarkLindblad MarkLindblad marked this pull request as ready for review January 27, 2025 22:03
@MarkLindblad MarkLindblad marked this pull request as draft January 27, 2025 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@MarkLindblad