Merge pull request #634 from instructlab/mergify/bp/release-v0.10/pr-628

uncap accelerate in `requirements-cuda.txt` (backport #628)

Merge pull request #546 from instructlab/mergify/bp/release-v0.10/pr-455

moves deepspeed requirements into their own file; add deepspeed extras (backport #455)

Merge pull request #528 from fynnsu/pylint-unused-argument

Enable pylint 'unused-argument' check

Merge pull request #518 from instructlab/mergify/bp/release-v0.10/pr-517

deps: Remove caps on ROCm dependencies (backport #517)

Merge pull request #489 from instructlab/mergify/bp/release-v0.10/pr-488

Change default internal sharding strategy to HYBRID_SHARD (backport #488)

Merge pull request #451 from instructlab/dependabot/github_actions/aw…

…s-actions/configure-aws-credentials-4.1.0

build(deps): Bump aws-actions/configure-aws-credentials from 4.0.2 to 4.1.0

Merge pull request #453 from JamesKunstle/rename-testing-dirs

change pytest targets. `test-unit` and `test-smoke` to `unit` and `smoke`

fix: model.forward now accepts return_dict via kwargs

Since huggingface/transformers#36794 it handles
the return_dict with can_return_tuple decorator that expects the
argument to be passed as kwarg.

Without the patch, instructlab functional tests fail with:

AttributeError: 'bool' object has no attribute 'unsqueeze'

This is because the passed return_dict argument is incorrectly
interpreted as cache_position.

Of course, the way the model forward method is overridden here is quite
problematic and would benefit from a refactor. This patch is a stop-gap
to fix the CI with minimal changes.

Signed-off-by: Ihar Hrachyshka <ihar.hrachyshka@gmail.com>

provide more docstrings

Signed-off-by: Oleg Silkin <97077423+RobotSail@users.noreply.github.com>

Merge pull request #407 from fabiendupont/hpu-remove-optimum-deps

Remove optimum dependency