Skip to content

Conversation

cyyever
Copy link
Collaborator

@cyyever cyyever commented Sep 9, 2023

The tests were disabled long before. It should be fine to enable them.
Fixes #35651, fixes #35636, fixes #35648.
cc @peterjc123 @mszhanyi @skyline75489 @nbcsm @vladimir-aubrecht @iremyux @Blackhex @cristianPanaite @malfet

@pytorch-bot
Copy link

pytorch-bot bot commented Sep 9, 2023

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/108930

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit e793d19 with merge base 8ff0036 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@cyyever
Copy link
Collaborator Author

cyyever commented Sep 9, 2023

@pytorchbot label "topic: not user facing"

@pytorch-bot pytorch-bot bot added the topic: not user facing topic category label Sep 9, 2023
@cyyever
Copy link
Collaborator Author

cyyever commented Sep 9, 2023

@pytorchbot label "topic: not user facing"

@cyyever
Copy link
Collaborator Author

cyyever commented Sep 9, 2023

@pytorchbot label ciflows/trunk

@pytorch-bot
Copy link

pytorch-bot bot commented Sep 9, 2023

Didn't find following labels among repository labels: ciflows/trunk

@cyyever
Copy link
Collaborator Author

cyyever commented Sep 9, 2023

@pytorchbot label ciflow/trunk

@pytorch-bot pytorch-bot bot added the ciflow/trunk Trigger trunk jobs on your pull request label Sep 9, 2023
@cyyever
Copy link
Collaborator Author

cyyever commented Sep 9, 2023

@pytorchbot label ciflow/binaries

@pytorch-bot pytorch-bot bot added the ciflow/binaries Trigger all binary build and upload jobs on the PR label Sep 9, 2023
@cyyever
Copy link
Collaborator Author

cyyever commented Sep 11, 2023

@pytorchbot rebase

@pytorchmergebot
Copy link
Collaborator

@pytorchbot started a rebase job onto refs/remotes/origin/viable/strict. Check the current status here

@pytorchmergebot
Copy link
Collaborator

Successfully rebased win_test onto refs/remotes/origin/viable/strict, please pull locally before adding more changes (for example, via git checkout win_test && git pull --rebase)

@drisspg drisspg added the triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module label Sep 11, 2023
@kit1980 kit1980 added the module: windows Windows support for PyTorch label Sep 12, 2023
:: Skip LibTorch tests when building a GPU binary and testing on a CPU machine
:: because LibTorch tests are not well designed for this use case.
if "%USE_CUDA%" == "0" IF NOT "%CUDA_VERSION%" == "cpu" exit /b 0
::if "%USE_CUDA%" == "0" IF NOT "%CUDA_VERSION%" == "cpu" exit /b 0
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wonder why is this commented out instead of removal?

if "%~1" == "module_test" goto :eof
:: See https://github.com/pytorch/pytorch/issues/25312
if "%~1" == "converter_nomigraph_test" goto :eof
:: See https://github.com/pytorch/pytorch/issues/35636
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please comment on #35636 if you think it's no longer an issue.

if "%~1" == "converter_nomigraph_test" goto :eof
:: See https://github.com/pytorch/pytorch/issues/35636
if "%~1" == "generate_proposals_op_gpu_test" goto :eof
:: See https://github.com/pytorch/pytorch/issues/35648
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please comment on #35648 if you think it's no longer an issue.

if "%~1" == "generate_proposals_op_gpu_test" goto :eof
:: See https://github.com/pytorch/pytorch/issues/35648
if "%~1" == "reshape_op_gpu_test" goto :eof
:: See https://github.com/pytorch/pytorch/issues/35651
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please comment on #35651 if you think it's no longer an issue.

@cyyever
Copy link
Collaborator Author

cyyever commented Sep 12, 2023

@pytorchmergebot merge

@pytorchmergebot
Copy link
Collaborator

Merge started

Your change will be merged once all checks pass (ETA 0-4 Hours).

Learn more about merging in the wiki.

Questions? Feedback? Please reach out to the PyTorch DevX Team

Advanced Debugging
Check the merge workflow status
here

@cyyever cyyever deleted the win_test branch September 13, 2023 01:40
pytorchmergebot pushed a commit that referenced this pull request Sep 26, 2023
Follows the work of #108930.

The commented test_custom_classes.py was removed since the file doesn't exist.
Pull Request resolved: #109847
Approved by: https://github.com/kit1980
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ciflow/binaries Trigger all binary build and upload jobs on the PR ciflow/trunk Trigger trunk jobs on your pull request Merged module: windows Windows support for PyTorch open source topic: not user facing topic category triaged This issue has been looked at a team member, and triaged and prioritized into an appropriate module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Caffe2 utility_ops_gpu_test fails on Windows Caffe2 ReshapeOpGPUTest crashes on Windows Caffe2 generate_proposals_op_gpu_test crashes on Windows

5 participants