Support windows-style path...?
Open
Nobody has claimed this yet.
documentation
good first issue
- Dominant language
- Python
- Stars
- 6
- Forks
- 5
- PR merge metrics
- No merged PRs in 30d
Description
We're not handling str path and just pass it to h5py.File here
But maybe we can wrap it with pathlib.Path to make sure it works...?
Since you copy + paste from windows finder, it gives you path with \
but users might not know of pathlib.Path, especially when you're using high-level interface like workflows.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start in src/scippnexus/file.py at lines 50-53 and inspect how string paths are passed to h5py.File. Reproduce the behavior with a Windows-style path copied as a string, then verify that the file-opening interface accepts it and that the existing test suite still passes.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100