kwarg fp16 doesn't exist anymore in Accelerate, replaced with mixed_precision

Philmod commented 1 year ago

Description

The fp16 kwarg was replaced by mixed_precision in Accelerate.

I fixed also a side bug with a requirement format that would gives me this error when pip install:

error in catalyst setup command: 'extras_require' must be a dictionary whose values are strings or lists of strings containing valid project/version requirement specifiers.

Type of Change

[ ] Examples / docs / tutorials / contributors update
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Improvement (non-breaking change which improves an existing feature)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

PR review

Anyone in the community is free to review the PR once the tests have passed. If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

CLAassistant commented 1 year ago

All committers have signed the CLA.

stale[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

catalyst-team / catalyst