Skip to content

[nnpack] create a new port#16344

Merged
vicroms merged 17 commits intomicrosoft:masterfrom
luncliff:port/nnpack
Apr 7, 2021
Merged

[nnpack] create a new port#16344
vicroms merged 17 commits intomicrosoft:masterfrom
luncliff:port/nnpack

Conversation

@luncliff
Copy link
Copy Markdown
Contributor

@luncliff luncliff commented Feb 21, 2021

What does your PR fix?

There was no port request for this project.

This is one of the 3rd party libraries for the PyTorch project. The PR will be used for future support of the libtorch port.

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

Looks like it supports Linux, Mac, iOS, and Android. The PR targets only 2 triplets.

  • x64-osx
  • x64-linux

The project doesn't have a configuration for Windows. (WIN32, AMD64, CMAKE_SYSTEM_NAME, etc.)

Does your PR follow the maintainer guide?

The library doesn't have a tag/release.

@luncliff luncliff changed the title Port/nnpack [nnpack] create a new port Feb 21, 2021
@luncliff luncliff marked this pull request as draft February 21, 2021 09:05
@luncliff
Copy link
Copy Markdown
Contributor Author

The port will be rebased after the merge of #16340 #16342.

@JackBoosY JackBoosY added category:new-port The issue is requesting a new library to be added; consider making a PR! depends:different-pr This PR or Issue depends on a PR which has been filed labels Feb 22, 2021
luncliff added 2 commits April 4, 2021 01:38
* there are no config for windows
There is a 'neon' backend, but it won't be considered for now
@luncliff luncliff marked this pull request as ready for review April 3, 2021 16:55
* use more correct expression
@JackBoosY JackBoosY removed the depends:different-pr This PR or Issue depends on a PR which has been filed label Apr 5, 2021
@JackBoosY
Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@JackBoosY
Copy link
Copy Markdown
Contributor

LGTM, already tested on Linux and OSX, curious why the CI status is N/A.

@JackBoosY JackBoosY added the info:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this. label Apr 5, 2021
@vicroms vicroms merged commit e1f1938 into microsoft:master Apr 7, 2021
@luncliff luncliff deleted the port/nnpack branch April 8, 2021 00:50
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:needs-maintainer-attention Lets the current 'on rotation' vcpkg maintainer know they need to look at this.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants