Skip to content

install optuna - #411

Closed
suk1yak1 wants to merge 1 commit into
Kaggle:masterfrom
suk1yak1:master
Closed

install optuna#411
suk1yak1 wants to merge 1 commit into
Kaggle:masterfrom
suk1yak1:master

Conversation

@suk1yak1

Copy link
Copy Markdown

Please add optuna :)
https://optuna.org/

@iwiwi

iwiwi commented Dec 20, 2018

Copy link
Copy Markdown

+1 from optuna dev team 😆

@rosbo rosbo added the new-package Requests for installing new packages label Dec 21, 2018
@rosbo

rosbo commented Dec 21, 2018

Copy link
Copy Markdown
Contributor

Hi,

Could you add a simple smoke test like this one: https://github.com/Kaggle/docker-python/blob/master/tests/test_fastai.py

This will prevent future regression.

Thank you

@toshihikoyanase

Copy link
Copy Markdown
Contributor

Hi @suk1yak1,

I'm a member of optuna dev team.
Although you may have already written the smoke test, we also wrote one. I'll share it just for your information.

import unittest

import optuna


class TestOptuna(unittest.TestCase):

    def test_study(self):

        def objective(trial):
            x = trial.suggest_uniform('x', -1., 1.)
            return x ** 2

        n_trials = 20
        study = optuna.create_study()
        study.optimize(objective, n_trials=n_trials)
        self.assertEqual(len(study.trials), n_trials)

If you do not have enough time, I'll take over this PR.
Thank you.

@rosbo

rosbo commented Feb 27, 2019

Copy link
Copy Markdown
Contributor

Closing in favor of #486

@rosbo rosbo closed this Feb 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

new-package Requests for installing new packages

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants