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

Enable cross-compiling for x86 + NVIDIA Grace (ARM64) #1049

Merged
merged 28 commits into from
Jul 19, 2023

Conversation

bashimao
Copy link
Contributor

@bashimao bashimao commented Jul 5, 2023

This fix build up on our previous fixes. It will bring back arm compiling, but this time we take a more wholesome approach.

Also, this change removes some redundancies that were in the build process.

Depends on
#1043

@bashimao bashimao requested review from jperez999 and EmmaQiaoCh July 5, 2023 15:23
@bashimao bashimao self-assigned this Jul 5, 2023
@bashimao bashimao added roadmap chore Infrastructure update labels Jul 5, 2023
@bashimao bashimao added this to the Merlin 23.07 milestone Jul 5, 2023
@github-actions
Copy link

github-actions bot commented Jul 5, 2023

Documentation preview

https://nvidia-merlin.github.io/Merlin/review/pr-1049

@bashimao bashimao requested a review from minseokl July 13, 2023 07:53
@bashimao
Copy link
Contributor Author

@jperez999 @EmmaQiaoCh @minseokl I wish this could be merged ;-)

@jperez999
Copy link
Collaborator

I am going to push this through as the failure does not have anything to do with the docker files edited in this PR. We will see if this runs successfully in our Pipeline and revert if we have any issues.

@jperez999 jperez999 merged commit cc863d7 into NVIDIA-Merlin:main Jul 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Infrastructure update roadmap
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants