♻️ use defaults for all params for full_model marker

[prashantgupta24]

Description

Use defaults for all params for full_model marker. This enables us to use full_model marker which defaults to TP 4 and CB along with using sendnn and the full model

Related Issues

[github-actions]

👋 Hi! Thank you for contributing to vLLM support on Spyre.
Just a reminder: Make sure that your code passes all the linting checks, otherwise your PR won't be able to be merged. To do so, first install the linting requirements, then run format.sh and commit the changes. This can be done with uv directly:

uv sync --frozen --group lint --active --inexact

Or this can be done with pip:

uv pip compile --group lint > requirements-lint.txt
pip install -r requirements-lint.txt
bash format.sh

Now you are good to go 🚀

[waleedqk]

bot:test
TEST_FILE=tests/e2e/test_spyre_basic.py MARKERS="spyre"

[waleedqk]

bot:test
TEST_FILE=tests/e2e/test_spyre_basic.py MARKERS="spyre"

[joerunde]

@waleedqk I think we'd need to run the full_model mark in order to test this change

[waleedqk]

@joerunde I was using an active PR to test the build and test process
not specifically testing the branch - but will change the marker so can check both

[waleedqk]

bot:test
TEST_FILE=tests/e2e/test_spyre_basic.py MARKERS="full_model"

[waleedqk]

bot:test
TEST_FILE=tests/e2e/test_spyre_basic.py MARKERS="full_model"

[waleedqk]

bot:test
TEST_FILE=tests/e2e/test_spyre_basic.py MARKERS="full_model"

[waleedqk]

bot:test
TEST_FILE=tests/e2e/test_spyre_basic.py MARKERS="full_model"

[waleedqk]

bot:test
TEST_FILE=tests/e2e/test_spyre_basic.py MARKERS="full_model"

[prashantgupta24]

Converting to a draft because we want to reimagine this with the vdpp marker that we are thinking about right now

[waleedqk]

bot:test
TEST_FILE=tests/e2e/test_spyre_basic.py

[waleedqk]

bot:test
TEST_FILE=tests/e2e/test_spyre_basic.py MARKERS="full_model"