Skip to content

Remove flask-pymongo and use only pymongo directly everywhere#1660

Draft
ml-evs wants to merge 2 commits intomainfrom
ml-evs/remove-flask-pymongo
Draft

Remove flask-pymongo and use only pymongo directly everywhere#1660
ml-evs wants to merge 2 commits intomainfrom
ml-evs/remove-flask-pymongo

Conversation

@ml-evs
Copy link
Copy Markdown
Member

@ml-evs ml-evs commented Mar 24, 2026

Something I've wanted to do for a while -- avoids duplicate connections to the database.

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 24, 2026

Codecov Report

❌ Patch coverage is 83.14607% with 30 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.38%. Comparing base (7e90e05) to head (aee07d9).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
pydatalab/src/pydatalab/file_utils.py 53.84% 6 Missing ⚠️
pydatalab/src/pydatalab/routes/v0_1/admin.py 81.81% 4 Missing ⚠️
pydatalab/src/pydatalab/routes/v0_1/blocks.py 78.94% 4 Missing ⚠️
pydatalab/src/pydatalab/routes/v0_1/info.py 20.00% 4 Missing ⚠️
pydatalab/src/pydatalab/routes/v0_1/auth.py 66.66% 3 Missing ⚠️
pydatalab/src/pydatalab/routes/v0_1/items.py 90.90% 3 Missing ⚠️
pydatalab/src/pydatalab/routes/v0_1/collections.py 88.23% 2 Missing ⚠️
pydatalab/src/pydatalab/login.py 91.66% 1 Missing ⚠️
pydatalab/src/pydatalab/routes/v0_1/export.py 90.90% 1 Missing ⚠️
pydatalab/src/pydatalab/routes/v0_1/files.py 80.00% 1 Missing ⚠️
... and 1 more
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1660   +/-   ##
=======================================
  Coverage   79.38%   79.38%           
=======================================
  Files          80       80           
  Lines        6940     6940           
=======================================
  Hits         5509     5509           
  Misses       1431     1431           
Files with missing lines Coverage Δ
pydatalab/src/pydatalab/apps/echem/blocks.py 79.20% <100.00%> (ø)
pydatalab/src/pydatalab/apps/xrd/blocks.py 90.42% <100.00%> (ø)
pydatalab/src/pydatalab/export.py 84.74% <100.00%> (ø)
pydatalab/src/pydatalab/main.py 64.22% <ø> (-0.65%) ⬇️
pydatalab/src/pydatalab/mongo.py 85.07% <ø> (-0.33%) ⬇️
pydatalab/src/pydatalab/permissions.py 89.32% <100.00%> (ø)
pydatalab/src/pydatalab/routes/v0_1/graphs.py 97.70% <100.00%> (ø)
pydatalab/src/pydatalab/routes/v0_1/groups.py 100.00% <100.00%> (ø)
pydatalab/src/pydatalab/scheduler.py 84.21% <100.00%> (ø)
pydatalab/src/pydatalab/versioning.py 87.71% <100.00%> (ø)
... and 11 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ml-evs ml-evs force-pushed the ml-evs/remove-flask-pymongo branch from 3b4f1b9 to 778197d Compare March 24, 2026 00:34
@cypress
Copy link
Copy Markdown

cypress Bot commented Mar 24, 2026

datalab    Run #4806

Run Properties:  status check passed Passed #4806  •  git commit 3c790be3a5 ℹ️: Merge aee07d9018acc6206f2c53101f282321df9daa8a into 7e90e05446240f3b0a9883e61e39...
Project datalab
Branch Review ml-evs/remove-flask-pymongo
Run status status check passed Passed #4806
Run duration 09m 32s
Commit git commit 3c790be3a5 ℹ️: Merge aee07d9018acc6206f2c53101f282321df9daa8a into 7e90e05446240f3b0a9883e61e39...
Committer Matthew Evans
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 488
View all changes introduced in this branch ↗︎

@ml-evs ml-evs force-pushed the ml-evs/remove-flask-pymongo branch from 9c76c56 to aee07d9 Compare April 20, 2026 22:56
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.

1 participant