search

autowarefoundation / modelzoo

A collection of machine-learned models for use in autonomous driving applications.
https://www.autoware.org/
Apache License 2.0
62 stars 24 forks source link

Add arm64 to CI #35

Closed ambroise-arm closed 3 years ago

ambroise-arm commented 3 years ago

Issue: https://github.com/autowarefoundation/modelzoo/issues/17

The use of the arm64 runner is to be tested, especially for the cuda build. If it doesn't work, it will need a follow-up patch to remove the feature.

Once the arm64 image is built a first time, another patch will be made to run pr-validation on it.

Build natively on amd64 and arm64, for both cpu and cuda; bundle and push the resulting images.

buildx is now used in the build script.

The dockerfiles have been merged into one per platform. This allows for:

JWhitleyWork commented 3 years ago

@ambroise-arm Sorry, I had the settings on the ARM runner set incorrectly. Should start the jobs now.

JWhitleyWork commented 3 years ago

@ambroise-arm The default image used on the ARM64 runner does not have Docker available. Let me look at the other runner and see if I did something differently.

LiyouZhou commented 3 years ago

@JWhitleyWork would appreciate if you could give the patch set a review. Thank you very much.

ambroise-arm commented 3 years ago

@JWhitleyWork Do you have an update on the arm64 runner?

JWhitleyWork commented 3 years ago

@JWhitleyWork Do you have an update on the arm64 runner?

Please see comment above on the image used.