🏗️ Enableln.connect() in lamindb

[falexwolf]

No description provided.

[github-actions]

🚀 Deployed on https://65e9fad1e9265841cb63f594--lamindb-setup-htry.netlify.app

[codecov]

Codecov Report

Attention: Patch coverage is 92.96875% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 80.89%. Comparing base (cac8551) to head (6aa18f0).
Report is 3 commits behind head on main.

Files	Patch %	Lines
lamindb_setup/_connect_instance.py	63.63%	4 Missing ⚠️
lamindb_setup/core/_settings.py	92.30%	2 Missing ⚠️
lamindb_setup/_check_setup.py	90.90%	1 Missing ⚠️
lamindb_setup/_init_instance.py	95.23%	1 Missing ⚠️
lamindb_setup/_schema.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #668      +/-   ##
==========================================
- Coverage   82.17%   80.89%   -1.29%     
==========================================
  Files          39       38       -1     
  Lines        2312     2340      +28     
==========================================
- Hits         1900     1893       -7     
- Misses        412      447      +35     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Koncopd]

setting cache should work for all instances, it sets the cache folder for all instances, not for the current one only, so it should not be commented out, if it is broken, then it is a genuine bug, i believe.

[falexwolf]

@Koncopd - I made an issue about this, it has nothing to do with the cache but between the impossibility of switching between instances: #669

I'm not entirely clear why ln_setup.close() enabled it before, it shouldn't have been possible because we currently can't switch between lamindb instances.

[falexwolf]

@Koncopd - this line makes it more rigorous because this measures whether django is setup or not.

[falexwolf]

This is why now, we need the _test flag. I will try this.

(Just confirming what you said on the call before!)

[falexwolf]

Tried it and it doesn't work because it doesn't attempt to access the SQLite file when run with _test. Let's see whether I can change this.

[Koncopd]

No, this should not matter, lock is called in _load_db, which is called always on load / connect.