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

fix(services/huggingface): Align with latest HuggingFace API #5123

Merged
merged 2 commits into from
Sep 18, 2024

Conversation

morristai
Copy link
Member

@morristai morristai commented Sep 17, 2024

Which issue does this PR close?

Closes #5122.

Rationale for this change

Example for latest response:

[
    {
        "type": "directory",
        "oid": "d564d0bc3dd917926892c55e3706cc116d5b165e",
        "size": 0,
        "path": "testdata/normal_dir",
        "lastCommit": {
            "id": "d836cf493b05f550c469d9c5cd553a3571372f17",
            "title": "Add testdata",
            "date": "2023-11-27T09:12:40.000Z"
        }
    },
    {
        "type": "directory",
        "oid": "d564d0bc3dd917926892c55e3706cc116d5b165e",
        "size": 0,
        "path": "testdata/special_dir  !@#$%^&()_+-=;',",
        "lastCommit": {
            "id": "d836cf493b05f550c469d9c5cd553a3571372f17",
            "title": "Add testdata",
            "date": "2023-11-27T09:12:40.000Z"
        }
    },
    {
        "type": "file",
        "oid": "9f37f432f4f882512be8e29529eb7b68a9ca7b3f",
        "size": 30482,
        "path": "testdata/normal_file.txt",
        "lastCommit": {
            "id": "d836cf493b05f550c469d9c5cd553a3571372f17",
            "title": "Add testdata",
            "date": "2023-11-27T09:12:40.000Z"
        },
        "security": {
            "blobId": "9f37f432f4f882512be8e29529eb7b68a9ca7b3f",
            "indexed": false,
            "safe": true,
            "avScan": {
                "virusFound": false,
                "virusNames": null
            },
            "pickleImportScan": null
        }
    },
    {
        "type": "file",
        "oid": "9f37f432f4f882512be8e29529eb7b68a9ca7b3f",
        "size": 30482,
        "path": "testdata/special_file  !@#$%^&()_+-=;',.txt",
        "lastCommit": {
            "id": "d836cf493b05f550c469d9c5cd553a3571372f17",
            "title": "Add testdata",
            "date": "2023-11-27T09:12:40.000Z"
        },
        "security": {
            "blobId": "9f37f432f4f882512be8e29529eb7b68a9ca7b3f",
            "indexed": false,
            "safe": true,
            "avScan": {
                "virusFound": false,
                "virusNames": null
            },
            "pickleImportScan": null
        }
    }
]

What changes are included in this PR?

Are there any user-facing changes?

Signed-off-by: Morris Tai <morristai01@gmail.com>
Signed-off-by: Morris Tai <morristai01@gmail.com>
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks a lot!

@Xuanwo Xuanwo merged commit f2c5739 into apache:main Sep 18, 2024
82 checks passed
@morristai morristai deleted the fix/huggingface_deserde_fail branch September 18, 2024 02:18
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bug: huggingface API Changed
2 participants