Skip to content

The tmol How To guide notebook - #399

Open
aleaverfay wants to merge 10 commits into
masterfrom
tmol_how_to_notebook
Open

The tmol How To guide notebook#399
aleaverfay wants to merge 10 commits into
masterfrom
tmol_how_to_notebook

Conversation

@aleaverfay

Copy link
Copy Markdown
Collaborator

We will need to flesh out this notebook as we document for users how to do things with Tmol

@codecov

codecov Bot commented Jul 17, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 74.38017% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.70%. Comparing base (a3e0864) to head (6b4c461).

Files with missing lines Patch % Lines
tmol/score/_score_function.py 65.43% 28 Missing ⚠️
tmol/relax/_fast_relax.py 70.00% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #399      +/-   ##
==========================================
- Coverage   90.74%   90.70%   -0.04%     
==========================================
  Files         445      445              
  Lines       36413    36463      +50     
==========================================
+ Hits        33042    33074      +32     
- Misses       3371     3389      +18     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

kierandidi added a commit that referenced this pull request Aug 8, 2026
Make every tutorial launchable on a GPU-backed Colab runtime and carry PR #399's supported scoring, sampling, and batched-relax concepts into executable, interactive examples.
@aleaverfay
aleaverfay requested review from fdimaio, jflat06 and kierandidi and removed request for fdimaio August 25, 2026 02:17
@aleaverfay

Copy link
Copy Markdown
Collaborator Author

We will probably not use any part of the notebook that I've created, but I do have several changes to the API that I didn't want getting lost. Let's merge this as it stands and continue on with the documentation refinement in its own branch.

@kierandidi

Copy link
Copy Markdown
Collaborator

Audited against current master and #465. The important ScoreFunction on/off-term fix has already been reimplemented in current code with regression tests (including shared multi-score-type terms and weight-tensor refresh). The notebook was already identified here as disposable. The remaining API patch should not be merged as-is: cartesian_fast_relax selects default_kin_min_fn when no minimizer is supplied, PoseStack.to only changes coords despite kernels being intentionally float32-oriented, and the eager top-level imports undo the current lazy-import design. This branch is conflicting and its old green check does not validate current CPU/CUDA/PyTorch combinations. I recommend closing it as superseded; any still-wanted convenience API should be proposed separately with current end-to-end tests.

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