Skip to content

PR8: fix(inference): review follow-up in pipeline scripts#33

Closed
SergeiNikolenko wants to merge 2 commits intopr7-worker-backendfrom
pr8-inference-review-fixes
Closed

PR8: fix(inference): review follow-up in pipeline scripts#33
SergeiNikolenko wants to merge 2 commits intopr7-worker-backendfrom
pr8-inference-review-fixes

Conversation

@SergeiNikolenko
Copy link
Copy Markdown
Member

Supersedes #25.

Summary

  • review follow-up fixes for inference pipeline scripts
  • restacked on top of new PR7 branch

Validation

  • uv run pytest -n0 -q tests/test_batch_clone_structure.py tests/test_euler_no_history.py tests/test_find_rigid_alignment_stability.py tests/test_predicted_transforms_no_protein_deepcopy.py tests/test_run_evaluation_accumulation.py tests/test_multigpu_cmd_build.py tests/test_multigpu_merge.py tests/test_multigpu_sharding.py tests/test_conformer_generation_batch.py tests/test_conformer_generation_in_memory.py tests/test_conformer_worker_contract.py

- Fix double comment marker (# # → #)
- Remove dead commented-out code and duplicate loop line
- Remove extra blank line before __main__ guard
@SergeiNikolenko
Copy link
Copy Markdown
Member Author

Closing this follow-up PR to keep the review queue focused on the remaining functional PRs.

@SergeiNikolenko SergeiNikolenko deleted the pr8-inference-review-fixes branch March 5, 2026 19:28
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