-
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
Pin the NumPy version with v1.23.5 in some images #2070
Pin the NumPy version with v1.23.5 in some images #2070
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: 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 |
Thanks @tenzen-y for quick fix. /lgtm |
We need to provide a clean fix in future when available |
Yes, I agree. We might be able to use the latest NumPy version once mxnet v2.0 or v1.9.2 is released. |
Since the errors are caused by mxnet codes, not by our training codes. |
Thank you for the prompt fix @tenzen-y ! |
It seems that E2E for bayesian-optimazation only failed...
https://github.com/kubeflow/katib/actions/runs/3767418520/jobs/6404930467#step:4:12390 I could not reproduce it on my local :( |
It looks like CI failure is caused by the skopt suggestion service image.
https://github.com/tenzen-y/katib/actions/runs/3768079424/jobs/6406216537#step:4:10644 |
ERROR:grpc._server:Exception calling application: module 'numpy' has no attribute 'int'
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/grpc/_server.py", line 443, in _call_behavior
response_or_iterator = behavior(argument, context)
File "/opt/katib/pkg/suggestion/v1beta1/skopt/service.py", line 55, in GetSuggestions
new_trials = self.base_service.getSuggestions(trials, request.current_request_number)
File "/opt/katib/pkg/suggestion/v1beta1/skopt/base_service.py", line 121, in getSuggestions
skopt_suggested = self.skopt_optimizer.ask(n_points=current_request_number)
File "/usr/local/lib/python3.9/site-packages/skopt/optimizer/optimizer.py", line 395, in ask
x = opt.ask()
File "/usr/local/lib/python3.9/site-packages/skopt/optimizer/optimizer.py", line 367, in ask
return self._ask()
File "/usr/local/lib/python3.9/site-packages/skopt/optimizer/optimizer.py", line 434, in _ask
return self.space.rvs(random_state=self.rng)[0]
File "/usr/local/lib/python3.9/site-packages/skopt/space/space.py", line 900, in rvs
columns.append(dim.rvs(n_samples=n_samples, random_state=rng))
File "/usr/local/lib/python3.9/site-packages/skopt/space/space.py", line 158, in rvs
return self.inverse_transform(samples)
File "/usr/local/lib/python3.9/site-packages/skopt/space/space.py", line 528, in inverse_transform
inv_transform = super(Integer, self).inverse_transform(Xt)
File "/usr/local/lib/python3.9/site-packages/skopt/space/space.py", line 168, in inverse_transform
return self.transformer.inverse_transform(Xt)
File "/usr/local/lib/python3.9/site-packages/skopt/space/transformers.py", line 309, in inverse_transform
X = transformer.inverse_transform(X)
File "/usr/local/lib/python3.9/site-packages/skopt/space/transformers.py", line 275, in inverse_transform
return np.round(X_orig).astype(np.int)
File "/usr/local/lib/python3.9/site-packages/numpy/__init__.py", line 284, in __getattr__
raise AttributeError("module {!r} has no attribute "
AttributeError: module 'numpy' has no attribute 'int' |
4b364ad
to
0a9812e
Compare
This skopt error is reported upstream, but the PR has not been merged... |
0a9812e
to
3c696f8
Compare
…uggestion service image Signed-off-by: Yuki Iwai <yuki.iwai.tz@gmail.com>
3c696f8
to
789bfc2
Compare
/lgtm |
Signed-off-by: Yuki Iwai yuki.iwai.tz@gmail.com
What this PR does / why we need it:
I pinned the NumPy version with v1.23.5 in the mxnet-mnist image and the skopt suggestion image to avoid the following errors:
/assign @andreyvelich @johnugeorge @gaocegege
Blocking: #2039 #2060 #2067
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: