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.

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

Issues while using clearml-data search CLI #1321

Open
abviv opened this issue Sep 6, 2024 · 5 comments
Open

Issues while using clearml-data search CLI #1321

abviv opened this issue Sep 6, 2024 · 5 comments
Labels
bug Something isn't working

Comments

@abviv
Copy link

abviv commented Sep 6, 2024

Describe the bug

A clear and concise description of what the bug is.
image
I am getting the above error while trying to use the clearml-data search command.

I am using clearml==1.16.4

To reproduce

Exact steps to reproduce the bug. Provide example code if possible.
Given

Expected behaviour

What is the expected behaviour? What should've happened but didn't?
This should've returned the proper list of datasets which i have already uploaded.

Environment

  • Server type (self hosted )
  • ClearML SDK Version
  • ClearML Server Version (Only for self hosted). Can be found on the bottom right corner of the settings screen. I am running the following verison:

image

  • Python Version = 3.12.2
  • OS ( Linux)

Related Discussion

If this continues a slack thread, please provide a link to the original slack thread.

@abviv abviv added the bug Something isn't working label Sep 6, 2024
@ainoam
Copy link
Collaborator

ainoam commented Sep 8, 2024

Thanks for reporting @abviv, we'll take a look.

@d-vignesh
Copy link
Contributor

Hi @ainoam , tested this bug out. It seems like the version of the dataset is None in some cases.
image

Adding a condition to assign version value as "" in this line when its None is fixing the issue. Would this fix be enough?
image

@abviv
Copy link
Author

abviv commented Sep 17, 2024

@d-vignesh Could you please create a new branch and do a pull request? I can test it locally.

@d-vignesh
Copy link
Contributor

@abviv can you please try the above branch and test.

@d-vignesh
Copy link
Contributor

@abviv did you get a chance to test this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants