Skip to content
This repository has been archived by the owner on Sep 18, 2024. It is now read-only.

Add comprehensive tests for tuners (merge into master) #1681

Merged
merged 6 commits into from
Nov 4, 2019

Conversation

ultmaster
Copy link
Contributor

No description provided.

@ultmaster ultmaster changed the title Add comprehensive tests for tuners Add comprehensive tests for tuners (merge into master) Oct 31, 2019
from nni.gridsearch_tuner.gridsearch_tuner import GridSearchTuner
from nni.hyperopt_tuner.hyperopt_tuner import HyperoptTuner
from nni.metis_tuner.metis_tuner import MetisTuner
from nni.smac_tuner.smac_tuner import SMACTuner
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a problem on windows: https://msrasrg.visualstudio.com/NNIOpenSource/_build/results?buildId=6298

======================================================================
ERROR: test_tuner (unittest.loader._FailedTest)
----------------------------------------------------------------------
ImportError: Failed to import test module: test_tuner
Traceback (most recent call last):
  File "C:\agent\_work\_tool\Python\3.6.4\x64\lib\unittest\loader.py", line 428, in _find_test_path
    module = self._get_module_from_name(name)
  File "C:\agent\_work\_tool\Python\3.6.4\x64\lib\unittest\loader.py", line 369, in _get_module_from_name
    __import__(name)
  File "C:\agent\_work\5\s\src\sdk\pynni\tests\test_tuner.py", line 34, in <module>
    from nni.smac_tuner.smac_tuner import SMACTuner
  File "C:\agent\_work\5\s\src\sdk\pynni\nni\smac_tuner\__init__.py", line 1, in <module>
    from .smac_tuner import SMACTuner
  File "C:\agent\_work\5\s\src\sdk\pynni\nni\smac_tuner\smac_tuner.py", line 29, in <module>
    from smac.facade.epils_facade import EPILS
ModuleNotFoundError: No module named 'smac'
----------------------------------------------------------------------
Ran 28 tests in 6.807s
FAILED
 (errors=1)

Since there is problem on pipeline itself, the status of the build is sucessful, it should be failed.

@chicm-ms
Copy link
Contributor

chicm-ms commented Nov 2, 2019

The Windows pipeline is ready, you can resolve the conflict and submit again

@QuanluZhang QuanluZhang merged commit 1f9b761 into master Nov 4, 2019
@liuzhe-lz liuzhe-lz deleted the dev-tuner-test branch November 6, 2019 18:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants