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
🚚 Move default storage location from lndb/
to .lamindb/
#796
Conversation
falexwolf
commented
Jun 22, 2023
•
edited
edited
- https://laminlabs.slack.com/archives/C04FPE8V01W/p1687432570162689
Codecov Report
@@ Coverage Diff @@
## main #796 +/- ##
==========================================
- Coverage 77.92% 77.90% -0.02%
==========================================
Files 31 31
Lines 1712 1715 +3
==========================================
+ Hits 1334 1336 +2
- Misses 378 379 +1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
lndb/
to .lamindb/
lamindb/_file_methods.py
Outdated
@@ -198,11 +198,15 @@ def inner(dir_path: Union[Path, UPath], prefix: str = "", level=-1): | |||
nonlocal files, directories | |||
if not level: | |||
return # 0, stop iterating | |||
stripped_dir_path = dir_path.as_posix().rstrip("/") | |||
# do not iterate through zarr directories | |||
if stripped_dir_path.endswith(".zarr"): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also needed for .zrad
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right, thanks for spotting this!