Skip to content

Update tp_utils tests - #23

Merged
xunyoyo merged 1 commit into
fastdeploy/model_executor/models/tp_utils.pyfrom
codex/add-copyright-header-and-clean-up-tests
Nov 24, 2025
Merged

xunyoyo merged 1 commit into
fastdeploy/model_executor/models/tp_utils.pyfrom
codex/add-copyright-header-and-clean-up-tests

Conversation

@xunyoyo

@xunyoyo xunyoyo commented Nov 24, 2025

Copy link
Copy Markdown
Owner

Summary

  • add the project Apache 2.0 license header to the tensor-parallel utilities test module
  • tighten the conversion_utils split/merge stub signature and adjust the related test call
  • remove the unnecessary coverage pragma from the main guard

Testing

  • pytest -c tests/cov_pytest.ini tests/model_executor/test_tp_utils.py
  • python -m coverage run -m pytest -c tests/cov_pytest.ini tests/model_executor/test_tp_utils.py
  • python -m coverage report -m --include='fastdeploy/model_executor/models/tp_utils.py'
  • pre-commit run --files tests/model_executor/test_tp_utils.py

Codex Task

@xunyoyo
xunyoyo merged commit d677963 into fastdeploy/model_executor/models/tp_utils.py Nov 24, 2025
@xunyoyo
xunyoyo deleted the codex/add-copyright-header-and-clean-up-tests branch November 26, 2025 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant