update-dockerfile #1531

rahul-tuli · 2023-04-21T13:46:54Z

This PR updates the dockerfile to use
CMD over ENTRYPOINT to allow easier overriding of commands for users

for example
Now the deepsparse.server can be run directly with one command

docker container run -it --gpus all ghcr.io/neuralmagic/sparseml:1.4.4-cu102 \
  sparseml.transformers.export_onnx --help

Replace ENTRYPOINT to CMD for easier overriding

e2a6af2

rahul-tuli requested review from bfineran, dbogunowicz, KSGulin, mgoin and mwitiderrick April 21, 2023 13:46

rahul-tuli self-assigned this Apr 21, 2023

rahul-tuli added enhancement New feature or request mle-team labels Apr 21, 2023

Merge branch 'main' into update-dockerfile

ac06d50

dbogunowicz approved these changes Apr 21, 2023

View reviewed changes

KSGulin approved these changes Apr 21, 2023

View reviewed changes

rahul-tuli merged commit a07df22 into main Apr 21, 2023

rahul-tuli deleted the update-dockerfile branch April 21, 2023 14:02

bfineran pushed a commit that referenced this pull request May 8, 2023

Replace ENTRYPOINT to CMD for easier overriding (#1531)

f6a7163

bfineran pushed a commit that referenced this pull request May 8, 2023

Replace ENTRYPOINT to CMD for easier overriding (#1531)

1fc7453

Provide feedback