✨ Track storage root location

[fredericenard]

Integrate

• https://github.com/laminlabs/lndb-hub/pull/10

[github-actions]

🚀 Deployed on https://62ffe59f37e9f82b8f6daf29--lamindb-qnwk.netlify.app

[codecov]

Codecov Report

Merging #137 (f3006c8) into main (ed3ba9c) will increase coverage by 0.63%.
The diff coverage is 94.64%.

❗ Current head f3006c8 differs from pull request most recent head 24f4a1e. Consider uploading reports for the commit 24f4a1e to get more accurate results

@@            Coverage Diff             @@
##             main     #137      +/-   ##
==========================================
+ Coverage   90.30%   90.93%   +0.63%     
==========================================
  Files          31       31              
  Lines         598      640      +42     
==========================================
+ Hits          540      582      +42     
  Misses         58       58              

Impacted Files	Coverage Δ
lamindb/_check_versions.py	69.23% <57.14%> (ø)
lamindb/datasets.py	100.00% <100.00%> (ø)
lamindb/dev/__init__.py	100.00% <100.00%> (ø)
lamindb/dev/_core.py	100.00% <100.00%> (ø)
lamindb/dev/db/_insert.py	93.00% <100.00%> (+0.81%)	⬆️
lamindb/do/_ingest.py	94.11% <100.00%> (+0.93%)	⬆️
lamindb/do/_query.py	86.17% <100.00%> (+0.70%)	⬆️
lamindb/schema/__init__.py	100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[fredericenard]

I don't understand why this happens, it's like the new version of lnschema_core was not used during setup. Locally everything works.

[falexwolf]

I think this error comes from the s3 bucket lamin0 already containing an existing lamindb instance that has not yet been migrated to the current schema. It likely prints out a warning somewhere, but it doesn't error (to allow the user to debug and manually migrate).

I'll remove the instance (sqlite file within bucket lamin0) and then run this again.

[falexwolf]

This worked! Now I just need to the current core schema version to the hub:

[falexwolf]

Worked!

Now one needs to update lndb-hub to the new schema:

[falexwolf]

I'll merge this for now so that we unblock downstream PRs.

We'll fix CI after.