-
Notifications
You must be signed in to change notification settings - Fork 5.5k
兼容mac的mps #2079
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
兼容mac的mps #2079
Conversation
|
@wzyyyqwzyyyq-maker please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
|
Thanks a lot for the contribution and for working on macOS MPS compatibility — it’s appreciated. That said, the PR currently includes a significant amount of unrelated and redundant content, which makes the changes difficult to review. For example:
Could you please clean up the PR by removing files and changes that are not directly related to macOS MPS compatibility, and keep the scope focused on what the title describes? This would make the review and potential merge much easier. Thanks again for your effort. |
I’ll clean up the PR to focus solely on macOS MPS compatibility and remove unrelated files and changes. I’ll update the scope to match the title and push a revised version shortly. |
Description
Motivation and Context
How Has This Been Tested?
pytest qlib/tests/test_all_pipeline.pyunder upper directory ofqlib.Screenshots of Test Results (if appropriate):
Types of changes