Skip to content

tfs:aarch64: add docker file to build aarch64 mkl configuration #1954

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

tfs:aarch64: add docker file to build aarch64 mkl configuration #1954

wants to merge 2 commits into from
+152 −0

Conversation

snadampal
Copy link
Contributor

This PR is dependent on #1953

@snadampal
Copy link
Contributor Author

I have signed the CLA, could you please rescan the PR?

@snadampal
Copy link
Contributor Author

Please let me know if I need to resubmit the PRs for the new CLA status to reflect..

@snadampal snadampal force-pushed the tfs_docker_mkl_aarch64 branch from edb4e4b to 9ca9a54 Compare January 20, 2022 19:49
@snadampal
Copy link
Contributor Author

Please let me know if there are any comments. I'm happy to address them.

mihaimaruseac
mihaimaruseac previously approved these changes Feb 3, 2022
View reviewed changes
@snadampal snadampal force-pushed the tfs_docker_mkl_aarch64 branch from 9ca9a54 to 75fa5e7 Compare July 15, 2022 21:42
@snadampal
Copy link
Contributor Author

the dependent PR #1953 got merged. Can this be merged as well? thank you!

@snadampal snadampal force-pushed the tfs_docker_mkl_aarch64 branch 2 times, most recently from 35908e8 to 8b1dc21 Compare August 2, 2022 15:11
@snadampal snadampal force-pushed the tfs_docker_mkl_aarch64 branch from 8b1dc21 to f8b8991 Compare August 2, 2022 16:44
@snadampal
Copy link
Contributor Author

the PR is marked ready for pull for quite sometime. Could someone please let me know what the next step is for it to get merged. Thank you!

@snadampal
Copy link
Contributor Author

Hi @mihaimaruseac , you had already approved this PR last year, appreciate if you could let me know the next steps for getting this merged.

@mihaimaruseac
Copy link
Contributor

I no longer work in TF, need someone from serving team to act on the repository and merge these PRs.

@snadampal
Copy link
Contributor Author

thanks, @mihaimaruseac . Can someone from serving team please review and merge this PR. Thank you!

@snadampal
Copy link
Contributor Author

pinging to see if someone can take a look at this PR, it's been open for more than 2 years.

@penpornk
Copy link
Member

@gharibian Could you please help review the PR?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mihaimaruseac mihaimaruseac mihaimaruseac left review comments

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
cla: yes ready to pull
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@snadampal @mihaimaruseac @penpornk @kokoro-team