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

[onnx-optimizer] create a new port #19258

Merged
merged 4 commits into from
Sep 16, 2021
Merged

[onnx-optimizer] create a new port #19258

merged 4 commits into from
Sep 16, 2021

Conversation

luncliff
Copy link
Contributor

What does your PR fix?

Create a new port for https://github.com/onnx/optimizer

Which triplets are supported/not supported? Have you updated the CI baseline?

Same as onnx #18073

  • x64-windows
  • x86-windows
  • x64-osx
  • x64-linux

Does your PR follow the maintainer guide?

Checked!

@luncliff
Copy link
Contributor Author

I will rebase this when #18073 is merged

@luncliff luncliff mentioned this pull request Jul 30, 2021
19 tasks
@JackBoosY JackBoosY added the category:new-port The issue is requesting a new library to be added; consider making a PR! label Aug 2, 2021
ports/onnx-optimizer/portfile.cmake Outdated Show resolved Hide resolved
ports/onnx/vcpkg.json Outdated Show resolved Hide resolved
ports/onnx-optimizer/vcpkg.json Outdated Show resolved Hide resolved
ports/onnx/fix-cmakelists.patch Outdated Show resolved Hide resolved
ports/onnx/portfile.cmake Outdated Show resolved Hide resolved
ports/onnx/portfile.cmake Outdated Show resolved Hide resolved
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for onnx but no changes to version or port version.
-- Version: 1.9.0
-- Old SHA: 5a0d95a1cf6502998635c8bdb58b1a5fbd416328
-- New SHA: bcfebbd5bb6d0072f56fb5166503522864636265
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for onnx but no changes to version or port version.
-- Version: 1.9.0
-- Old SHA: 5a0d95a1cf6502998635c8bdb58b1a5fbd416328
-- New SHA: bcfebbd5bb6d0072f56fb5166503522864636265
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***
Error: Local changes detected for onnx-optimizer but no changes to version or port version.
-- Version: 0.2.6
-- Old SHA: 0bf85a2b4276beb2a541549955510e4dfdb85dc2
-- New SHA: 644473963518056d45a410b9658565affdcc8ecb
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for onnx but no changes to version or port version.
-- Version: 1.9.0
-- Old SHA: b53c9c9e969928def925c57ea5ddcdfb09293693
-- New SHA: 43475433efc6966a156018fcc698cb74357f1613
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***
After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 21935b744dc411e0fb1926618fdfb18c67d4b188 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 2028c11..fc92276 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -4584,6 +4584,10 @@
       "baseline": "1.9.0",
       "port-version": 0
     },
+    "onnx-optimizer": {
+      "baseline": "0.2.6",
+      "port-version": 0
+    },
     "onnxruntime-gpu": {
       "baseline": "1.5.1",
       "port-version": 0

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for onnx but no changes to version or port version.
-- Version: 1.9.0
-- Old SHA: b53c9c9e969928def925c57ea5ddcdfb09293693
-- New SHA: 43475433efc6966a156018fcc698cb74357f1613
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***
After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 21935b744dc411e0fb1926618fdfb18c67d4b188 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/baseline.json b/versions/baseline.json
index 2028c11..fc92276 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -4584,6 +4584,10 @@
       "baseline": "1.9.0",
       "port-version": 0
     },
+    "onnx-optimizer": {
+      "baseline": "0.2.6",
+      "port-version": 0
+    },
     "onnxruntime-gpu": {
       "baseline": "1.5.1",
       "port-version": 0

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for onnx but no changes to version or port version.
-- Version: 1.9.0
-- Old SHA: b53c9c9e969928def925c57ea5ddcdfb09293693
-- New SHA: 43475433efc6966a156018fcc698cb74357f1613
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***
After committing all other changes, the version database must be updated
git add -u && git commit
git checkout 21935b744dc411e0fb1926618fdfb18c67d4b188 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/o-/onnx.json b/versions/o-/onnx.json
index 4efd7e2..5f4fc6e 100644
--- a/versions/o-/onnx.json
+++ b/versions/o-/onnx.json
@@ -1,7 +1,7 @@
 {
   "versions": [
     {
-      "git-tree": "43475433efc6966a156018fcc698cb74357f1613",
+      "git-tree": "b53c9c9e969928def925c57ea5ddcdfb09293693",
       "version-semver": "1.9.0",
       "port-version": 0
     }

ports/onnx/fix-pybind11.patch Outdated Show resolved Hide resolved
* migrate changes to new branch
* ONNX_USE_PROTOBUF_SHARED_LIBS is for Windows
@luncliff
Copy link
Contributor Author

luncliff commented Sep 10, 2021

Tried git rebase master, but the result was dirty. Moved the changes for simplicity.

@luncliff luncliff requested a review from JackBoosY September 10, 2021 15:56
ports/onnx-optimizer/portfile.cmake Outdated Show resolved Hide resolved
ports/onnx-optimizer/portfile.cmake Outdated Show resolved Hide resolved
@JackBoosY JackBoosY added info:reviewed Pull Request changes follow basic guidelines and removed requires:author-response labels Sep 16, 2021
@BillyONeal BillyONeal merged commit 2aa021c into microsoft:master Sep 16, 2021
@BillyONeal
Copy link
Member

Thanks for the new port!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:new-port The issue is requesting a new library to be added; consider making a PR! info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants