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

Fix 2 issues #5779

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

TimSchim
Copy link

@TimSchim TimSchim commented May 8, 2024

Description

HPO with GPTuner is not working with current versions of SciPy

Issue 1:

Starting from SciPy v1.11.0 the shape of argument x0 for scipy.optimize.minimize() should be of shape (n,). The current NNI implementation assumes shape (1, n).
Related issues:

Issue 2:

scipy.optimize.minimize() returns a scipy.optimize.OptimizeResult object. The current NNI implementation assumes the "fun" item to be an array, when the type should actually be a float.
Related issues:

  • #4978 (closed but not fixed)

Test Options

  • fast test
  • full test - HPO
  • full test - NAS
  • full test - compression

Checklist

  • test case
  • doc

How to test

 - x0 argument of scipy.optimize.minimize function expects 1D-array after version 1.11.0
 - res.fun is a scalar and not an array
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant