Skip to content

Uv setup mchip#434

Closed
paulsonak wants to merge 39 commits intouv-setupfrom
uv-setup-mchip
Closed

Uv setup mchip#434
paulsonak wants to merge 39 commits intouv-setupfrom
uv-setup-mchip

Conversation

@paulsonak
Copy link
Copy Markdown
Collaborator

OK @mauvais2 the rebasing/merging was confusing, but the most recent commit ec3076c has the new uv lock file for mchip. if you want, instead of merging this branch you can just do git cherry pick ec3076cb14c647e55d9b976442e893a2a349a8b3 and pull the lock file to your clean branch.

It seems like in my branch, there are the same code changes you have but with different commit hashes from different rebases. so, it. might be easier to do the git cherry pick.

mauvais2 and others added 30 commits April 7, 2026 14:04
…hyperopt.ipynb) hyperopt -> Optuna (step 13)
@paulsonak paulsonak requested a review from mauvais2 April 17, 2026 22:20
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 18, 2026

Codecov Report

❌ Patch coverage is 93.45794% with 7 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
atomsci/ddm/utils/hyperparam_search_wrapper.py 93.33% 7 Missing ⚠️
@@             Coverage Diff              @@
##           uv-setup     #434      +/-   ##
============================================
- Coverage     51.38%   49.68%   -1.71%     
============================================
  Files            36       37       +1     
  Lines         10690    11714    +1024     
============================================
+ Hits           5493     5820     +327     
- Misses         5197     5894     +697     
Flag Coverage Δ
unittests 49.68% <93.45%> (-1.71%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
atomsci/ddm/__init__.py 100.00% <ø> (ø)
atomsci/ddm/pipeline/model_wrapper.py 68.33% <100.00%> (+2.04%) ⬆️
atomsci/ddm/pipeline/parameter_parser.py 92.54% <100.00%> (+1.23%) ⬆️
atomsci/ddm/utils/hyperparam_search_wrapper.py 29.30% <93.33%> (ø)
🚀 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.

@mauvais2
Copy link
Copy Markdown
Contributor

Commit ec3076c was merged/included in to uv-setup branch.

@mauvais2 mauvais2 closed this Apr 20, 2026
mauvais2 added a commit that referenced this pull request Apr 23, 2026
@mauvais2 mauvais2 deleted the uv-setup-mchip branch April 24, 2026 20:13
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.

2 participants