Skip to content
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

Revive static_runtime_benchmark build and test #87660

Closed

Conversation

huydhn
Copy link
Contributor

@huydhn huydhn commented Oct 25, 2022

This build uses the wrong BUILD_ENVIRONMENT pytorch-linux-focal-py3, thus it hasn't been run for a long time (forgotten). The name was probably the old name of the build environment we used in the past. The convention today doesn't have the pytorch- prefix. There is a TODO for this:

TODO: this condition is never (BUILD_ENVIRONMENT doesn't start with pytorch-), need to fix this.

This is done as part of T131829540, where we want
static_runtime_benchmark build and test jobs to run in OSS CI to avoid breaking internal

@huydhn huydhn self-assigned this Oct 25, 2022
@pytorch-bot
Copy link

pytorch-bot bot commented Oct 25, 2022

🔗 Helpful Links

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

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

✅ No Failures

As of commit 9654ddf:
💚 Looks good so far! There are no failures yet. 💚

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

@pytorch-bot pytorch-bot bot added the release notes: releng release notes category label Oct 25, 2022
@huydhn huydhn added test-config/default ciflow/trunk Trigger trunk jobs on your pull request and removed release notes: releng release notes category labels Oct 25, 2022
@pytorch-bot pytorch-bot bot added the release notes: releng release notes category label Oct 25, 2022
@huydhn huydhn changed the title Fix static_runtime_benchmark build Revive static_runtime_benchmark build Oct 25, 2022
@huydhn huydhn requested review from kit1980, malfet, mehtanirav and a team October 25, 2022 03:13
@huydhn huydhn marked this pull request as ready for review October 25, 2022 03:14
@huydhn huydhn marked this pull request as draft October 25, 2022 09:45
@huydhn
Copy link
Contributor Author

huydhn commented Oct 26, 2022

@pytorchbot rebase

@pytorchmergebot
Copy link
Collaborator

@pytorchbot successfully started a rebase job. Check the current status here

This build uses the wrong BUILD_ENVIRONMENT and hasn't been run for a
long time
@pytorchmergebot
Copy link
Collaborator

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

@pytorchmergebot pytorchmergebot force-pushed the fix-status-runtime-benchmark-build branch from 18a9a6f to 9f8ad25 Compare October 26, 2022 02:53
@huydhn huydhn marked this pull request as ready for review October 28, 2022 01:54
@huydhn huydhn changed the title Revive static_runtime_benchmark build Revive static_runtime_benchmark build and test Oct 28, 2022
benchmarks/static_runtime/test_utils.cc Outdated Show resolved Hide resolved
@huydhn huydhn force-pushed the fix-status-runtime-benchmark-build branch from 10b1f96 to 34da8e1 Compare November 1, 2022 18:12
@huydhn
Copy link
Contributor Author

huydhn commented Nov 8, 2022

@pytorchbot 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

@pytorchmergebot
Copy link
Collaborator

Merge failed

Reason: New commits were pushed while merging. Please rerun the merge command.

Details for Dev Infra team Raised by workflow job

@huydhn
Copy link
Contributor Author

huydhn commented Nov 8, 2022

@pytorchbot 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

@huydhn huydhn deleted the fix-status-runtime-benchmark-build branch November 17, 2022 02:56
kulinseth pushed a commit to kulinseth/pytorch that referenced this pull request Dec 10, 2022
This build uses the wrong BUILD_ENVIRONMENT `pytorch-linux-focal-py3`, thus it hasn't been run for a long time (forgotten). The name was probably the old name of the build environment we used in the past.  The convention today doesn't have the `pytorch-` prefix. There is a TODO for this:

> TODO: this condition is never (BUILD_ENVIRONMENT doesn't start with pytorch-), need to fix this.

This is done as part of [T131829540](https://www.internalfb.com/intern/tasks/?t=131829540), where we want
 `static_runtime_benchmark` build and test jobs to run  in OSS CI to avoid breaking internal

* I also fix some compiler warning errors `-Werror=sign-compare`, `-Werror,-Wunused-const-variable`, and gcc7 compatibility issue along the way because this hasn't been run for a long time.
* Reviving this test also reveals a small bug in `PrepackWeights` test in `test_static_runtime.cc` added recently in pytorch#85289. The test refers to an internal ops and should only be run internally. This has been fixed by pytorch#87799 (To be merged)
Pull Request resolved: pytorch#87660
Approved by: https://github.com/malfet
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ciflow/trunk Trigger trunk jobs on your pull request Merged release notes: releng release notes category test-config/default
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants