Issue running the base example on Windows

[EmilNichita]

Polaris version

0.9.1

Python Version

3.11

Operating System

Windows

Installation

Using pip into a conda env

Description

Running the first example in the docs, after successfully logging in, gives an error on a Windows 11 machine.

Steps to reproduce

import polaris as po

dataset = po.load_dataset("polaris/hello-world")

File [~\.conda\envs\molgp\Lib\site-packages\polaris\hub\storage.py:74](http://localhost:8888/lab/tree/notebooks/~/.conda/envs/polaris-test/Lib/site-packages/polaris/hub/storage.py#line=73), in S3Store.__init__(self, path, access_key, secret_key, token, endpoint_url, part_size, content_type)
      63 def __init__(
      64     self,
      65     path: str,
    (...)
      71     content_type: str = "application/octet-stream",
      72 ) -> None:
 ---> 74     bucket_name, prefix = path.split("/", 1)
      77     self.s3_client = boto3.client(
      78         "s3",
      79         aws_access_key_id=access_key,
    (...)
      82         endpoint_url=endpoint_url,
      83     )
      84     self.bucket_name = bucket_name
 
 ValueError: not enough values to unpack (expected 2, got 1)

Adding a print statement to the path variable, gives path=polaris\dataset\polaris\hello-world. Windows/Unix path differences strike again :( .

Quick fix suggestion for enabling Windows users: use proper handling of paths (built-in path library, one of the dozens local&cloud path handlers).

Looking forward to trying the library again once this is solved!

Additional output

No response

[cwognum]

Thanks for reporting this, @EmilNichita ! We'll look into it.

We should also look into running the test suite on Windows through the Github CICD. Most maintainers of Polaris work on Unix, so Windows is a bit of a blind spot. We also don't have integration tests setup yet for the client-hub communication, but now that #221 is merged this is something we should do as well.

[cwognum]

Hey @EmilNichita, it should be fixed with #228 . We also added Windows to our CICD and added an integration test with the Hub, so we'll spot this sooner next time. Thanks again for raising this! I'll release a new Polaris version soon!

[cwognum]

Just released 0.9.3 with the fix. Let me know if you run into any other issues. Thanks again! 👍

[EmilNichita]

Can confirm the 0.9.3 version works as expected, thanks again for solving this so quickly!