-
Notifications
You must be signed in to change notification settings - Fork 443
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
Fix kubeflowkatib/mxnet-mnist image #1866
Conversation
When I use Downloading data/train-labels-idx1-ubyte.gz from https://repo.mxnet.io/gluon/dataset/mnist/train-labels-idx1-ubyte.gz...
Downloading data/train-images-idx3-ubyte.gz from https://repo.mxnet.io/gluon/dataset/mnist/train-images-idx3-ubyte.gz...
Downloading data/t10k-labels-idx1-ubyte.gz from https://repo.mxnet.io/gluon/dataset/mnist/t10k-labels-idx1-ubyte.gz...
Downloading data/t10k-images-idx3-ubyte.gz from https://repo.mxnet.io/gluon/dataset/mnist/t10k-images-idx3-ubyte.gz...
[17:14:13] ../src/storage/storage.cc:202: Using Pooled (Naive) StorageManager for CPU
Traceback (most recent call last):
File "/opt/mxnet-mnist/mnist.py", line 86, in <module>
fit.fit(args, sym, get_mnist_iter)
File "/opt/mxnet-mnist/common/fit.py", line 228, in fit
model = mx.mod.Module(
AttributeError: module 'mxnet' has no attribute 'mod' |
It seems to fail to deploy katib in
https://github.com/kubeflow/katib/runs/6475581693?check_suite_focus=true#step:6:372 Could you help to resolve this problem? @DomFleischmann @knkski @ca-scribner |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
I don't recognise this error, but I'll get someone to take a look. Should be a quick fix and likely is related to some tooling and not actually broken by this PR. |
Waiting on CI to confirm but this should be fixed by #1867. |
Thank you for letting me know @ca-scribner. |
Please rebase ! |
@johnugeorge I have rebased this branch. |
Thanks @tenzen-y /lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: johnugeorge, tenzen-y The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What this PR does / why we need it:
It seems that
mxnet/python:latest_cpu_native_py3
image has been deleted from upstream dockerhub.As a result, our suite test can no longer be run as shown below, so I changed the base image from
mxnet/python:latest_cpu_native_py3
topython:3.9
.Argo Workflow Logs
Also, I have used
ytenzen/katib-mxnet-mnist:fix-mxnet-image
to check the operation of the new image.Which issue(s) this PR fixes (optional, in
fixes #<issue number>(, fixes #<issue_number>, ...)
format, will close the issue(s) when PR gets merged):Fixes #
Checklist: