Skip to content

Comments

Fix FileFeatureStorage path resolution on case-sensitive filesystems#2135

Open
pdwi2020 wants to merge 1 commit intomicrosoft:mainfrom
pdwi2020:codex/filefeaturestorage-case-sensitive
Open

Fix FileFeatureStorage path resolution on case-sensitive filesystems#2135
pdwi2020 wants to merge 1 commit intomicrosoft:mainfrom
pdwi2020:codex/filefeaturestorage-case-sensitive

Conversation

@pdwi2020
Copy link

@pdwi2020 pdwi2020 commented Feb 15, 2026

Summary

  • keep the existing lowercase path as default for feature storage
  • add case-aware fallback that resolves existing instrument directories/files by case-insensitive match
  • add a regression test for uppercase instrument directory (ABB-U) with close.day.bin

Why

This fixes loading failures on case-sensitive filesystems when data is stored under uppercase instrument folders, as reported in #2053.

Fixes #2053

Testing

  • python3 -m py_compile qlib/data/storage/file_storage.py tests/storage_tests/test_storage.py
  • Full pytest target requires compiled qlib C extensions in this environment.

@pdwi2020
Copy link
Author

@microsoft-github-policy-service agree

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

FileFeatureStorage lowercases the instrument directory name when building feature paths

1 participant