Skip to content

Comments

feat: Add from_uri support for all services#6668

Merged
Xuanwo merged 10 commits intomainfrom
registry-more
Oct 16, 2025
Merged

feat: Add from_uri support for all services#6668
Xuanwo merged 10 commits intomainfrom
registry-more

Conversation

@Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Oct 14, 2025

Which issue does this PR close?

Part of #5445

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

@Xuanwo Xuanwo requested a review from PsiACE October 14, 2025 14:12
@dosubot dosubot bot added size:XXL This PR changes 1000+ lines, ignoring generated files. releases-note/feat The PR implements a new feature or has a title that begins with "feat" labels Oct 14, 2025
Signed-off-by: Xuanwo <[email protected]>
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 15, 2025
Signed-off-by: Xuanwo <[email protected]>
Signed-off-by: Xuanwo <[email protected]>
Signed-off-by: Xuanwo <[email protected]>
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. and removed size:XXL This PR changes 1000+ lines, ignoring generated files. labels Oct 16, 2025
@Xuanwo Xuanwo merged commit a0aa222 into main Oct 16, 2025
399 of 400 checks passed
@Xuanwo Xuanwo deleted the registry-more branch October 16, 2025 12:56
def __init__(
self,
scheme: Literal["aliyun_drive"],
scheme: Literal["aliyun-drive"],
Copy link
Contributor

Choose a reason for hiding this comment

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

@Xuanwo aren't some changes required in dev/src/generate/python.j2 as well. otherwise these will be overwritten by just generate python

Copy link
Member Author

Choose a reason for hiding this comment

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

You are right on this! Would you like to have a PR for this? Or I'll create one as follow up.

Copy link
Contributor

Choose a reason for hiding this comment

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

let me take a shot at this in the next one.

#[test]
fn from_uri_sets_repo_type_id_and_revision() {
let uri = OperatorUri::new(
"huggingface://model/opendal/sample/main/dataset",
Copy link

@lhoestq lhoestq Nov 10, 2025

Choose a reason for hiding this comment

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

Hi ! Just a heads up that HF uses hf://{owner}/{repo}/{path_in_repo}, it would be cool to follow this :)

And to summarize the spec, if there is a revision it should be hf://{owner}/{repo}@{revision}/{path_in_repo}

and for datasets and spaces it's respectively hf://datasets/{owner}/{repo}/{path_in_repo} and hf://spaces/{owner}/{repo}/{path_in_repo}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer releases-note/feat The PR implements a new feature or has a title that begins with "feat" size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants