SDK task.upload_data() cannot be used with resources of the Path type

[zhiltsov-max]

Actions before raising this issue

• I searched the existing issues and did not find anything similar.
• I read/searched the docs

Steps to Reproduce

SDK has a Task.upload_data(), which claims to support StrPath-typed resources, but it is actually only works for LOCAL resource_type.

cvat/cvat-sdk/cvat_sdk/core/proxies/tasks.py

Line 73 in fec040d

resources: Sequence[StrPath],

Expected Behavior

Task should be created.

Possible Solution

Remove invalid error here:

cvat/cvat-sdk/cvat_sdk/core/proxies/tasks.py

Lines 112 to 114 in fec040d

	for resource in resources:
	if not isinstance(resource, str):
	raise TypeError(f"resources: expected instances of str, got {type(resource)}")

, replace it with a type conversion.

Context

No response

Environment

[Eslam-Radwan]

what should I do to face the bug

and what does this line print when the bug happens

raise TypeError(f"resources: expected instances of str, got {type(resource)}")

[zhiltsov-max]

Ax example:

from pathlib import PurePosixPath
from cvat_sdk import make_client, models
from cvat_sdk.core.proxies.tasks import ResourceType


with make_client("http://localhost", port=8080, credentials=("user", "pass")) as client:
    task = client.tasks.create_from_data(
        spec=models.TaskWriteRequest(
            name="image task with cs",
            labels=[{"name": "cat"}],
        ),
        resources=[PurePosixPath("image1.png"), PurePosixPath("image2.png")],
        resource_type=ResourceType.SHARE,
        data_params=dict(
            cloud_storage_id=your_cloud_storage_id,
            image_quality=70,
        ),
    )

[Eslam-Radwan]

can't we just replace

from _typeshed import StrPath, SupportsWrite

with

from _typeshed import SupportsWrite
from os import PathLike

[zhiltsov-max]

Doing so won't change the error. The function still fails when Path-s are passed.

[RavinduWeerakoon]

So, we need to add logic for type conversions inside the for loop, such as converting Posix to string, right? If that's the case, I'd love to take care of it!

[RavinduWeerakoon]

So, we need to add logic for type conversions inside the for loop, such as converting Posix to string, right? If that's the case, I'd love to take care of it!

So can you assign this issue to me. I have set the development environment according to the guidelines. And I guess there won't be an issue with the os.path since it is already treated as strings