Skip to content

refactor(inductive): remove TSVM implementation and prune obsolete inductive TSVM experiments#37

Merged
melvinbarbaux merged 1 commit intomainfrom
remove-inductive-tsvm
Feb 17, 2026
Merged

refactor(inductive): remove TSVM implementation and prune obsolete inductive TSVM experiments#37
melvinbarbaux merged 1 commit intomainfrom
remove-inductive-tsvm

Conversation

@melvinbarbaux
Copy link
Copy Markdown
Collaborator

Summary

What does this PR change?

Checklist

  • Tests added or updated
  • CI green
  • Changelog updated
  • Documentation updated

Notes

Anything reviewers should know?

@melvinbarbaux melvinbarbaux self-assigned this Feb 17, 2026
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review


P2 Badge Keep TSVM by-method command file for transductive runs

Deleting by_method/tsvm.md drops all transductive TSVM commands from the by-method command set even though TSVM is still implemented and registered for transductive mode (src/modssc/transductive/registry.py) and transductive TSVM configs still exist under bench/configs/experiments/best/*/transductive/tsvm. This also leaves bench/configs/experiments/best/commands/02_by_method.md:53 pointing to a missing file, so any workflow that follows the by-method index will skip or fail on TSVM experiments.

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@melvinbarbaux melvinbarbaux merged commit 8911893 into main Feb 17, 2026
9 checks passed
@melvinbarbaux melvinbarbaux deleted the remove-inductive-tsvm branch February 17, 2026 16:55
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