SMAC3 icon indicating copy to clipboard operation
SMAC3 copied to clipboard

Implement NormalIntegerHyperparameter, NormalFloatHyperparameter and other distributions

Open KOLANICH opened this issue 6 years ago • 10 comments

Efficiently. I already have an inefficient implementation, feel free to reuse it, but it is damn inefficient (probes the same points multiple times).

KOLANICH avatar Jan 02 '19 21:01 KOLANICH

Dear @KOLANICH ,

Thank you for this proposal. It is also on our agenda for quite some time, but we have not implemented it so far.

Since this feature request does not belong to our SMAC package but to our configspace (https://github.com/automl/ConfigSpace), I will close this ticket here.

mlindauer avatar Jan 04 '19 19:01 mlindauer

Since this feature request does not belong to our SMAC package, but to our configspace .

They seem to be already implemented there (at least Normal), aren't they? It is SMAC which doesn't support them.

KOLANICH avatar Jan 04 '19 19:01 KOLANICH

@mfeurer could you please reply to it.

mlindauer avatar Jan 04 '19 19:01 mlindauer

@KOLANICH is correct, they are implemented in the ConfigSpace package but not in the SMAC package.

mfeurer avatar Jan 07 '19 08:01 mfeurer

@mfeurer could you please remind of the reason why it is not implemented in SMAC?

mlindauer avatar Jan 07 '19 08:01 mlindauer

  • It is not implemented in the original SMAC package and not compatible with the pcs format.
  • So far no one was interested in it.

I'd be happy to review pull requests on this, though.

mfeurer avatar Jan 07 '19 08:01 mfeurer

@KOLANICH we will consider this request for our next SMAC release.

mlindauer avatar Jan 09 '19 08:01 mlindauer

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Aug 09 '22 11:08 stale[bot]

Since we drop the pcs format in v2.0 we could integrate this feature too. What do you think @mfeurer?

renesass avatar Aug 12 '22 12:08 renesass

That sounds reasonable. Similarly, the beta-distributed hyperparameters could be used.

mfeurer avatar Aug 12 '22 14:08 mfeurer

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

stale[bot] avatar Oct 11 '22 22:10 stale[bot]

Hi, we support the following hyperparameter types now:

UniformFloatHyperparameter,
UniformIntegerHyperparameter,
OrdinalHyperparameter,
CategoricalHyperparameter,
NormalFloatHyperparameter,
NormalIntegerHyperparameter,
BetaFloatHyperparameter,
BetaIntegerHyperparameter,

.However, NormalFloatHyperparameter, NormalIntegerHyperparameter, BetaFloatHyperparameter, BetaIntegerHyperparameter cannot be inactive, i.e. they cannot be used with conditions as for now.

benjamc avatar Mar 29 '23 09:03 benjamc

Thanks to everyone who was involved in implementing this :+1:. I cannot test it right now because I'm a bit occupied with other urgent work, so no ETA when the SMAC3 backend for UniOpt is ready.

KOLANICH avatar Mar 29 '23 15:03 KOLANICH