Skip to content

Conversation

@Sameena-Thabassum
Copy link
Collaborator

No description provided.

Copy link
Owner

@srisudarsan srisudarsan left a comment

Choose a reason for hiding this comment

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

Only typing seems to be wrong, rest looks good

overlap_all: bool
starting_page_number: Optional[int] = None
include_slide_notes: bool
custom_metadata: Optional[str] = None
Copy link
Owner

Choose a reason for hiding this comment

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

is this string ?

),
] = True,
custom_metadata: Annotated[
Optional[str],
Copy link
Owner

Choose a reason for hiding this comment

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

this should be dict then ?

--workers $WORKERS"

if [[ -n "$OPTIONAL_TIMEOUT" ]]; then
$OPTIONAL_TIMEOUT $CMD
Copy link
Owner

Choose a reason for hiding this comment

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

should we not be doing eval here too ?

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.

3 participants