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

Priorband: Defining default value + default_confidence for fidelity parameter #101

Open
danrgll opened this issue May 23, 2024 · 3 comments
Labels
question Further information is requested

Comments

@danrgll
Copy link
Contributor

danrgll commented May 23, 2024

As far as I understand Priorband, it doesn’t make sense to have this functionality and should raise an error.

pipeline_space:
  learning_rate:
    lower: 1e-5
    upper: 1e-1
    log: True  # Log scale for learning rate
    default: 1e-2
    default_confidence: "medium"
  epochs:
    lower: 5
    upper: 20
    is_fidelity: True
    **default: 10**
    **default_confidence: "medium"**
  dropout_rate:
    lower: 0.1
    upper: 0.5
    default: 0.2
    default_confidence: "high"
  optimizer:
    choices: [adam, sgd, adamw]
    default: adam
    # default confidence low
  batch_size: 64
@danrgll danrgll added the question Further information is requested label May 23, 2024
@Neeratyoy
Copy link
Contributor

What exactly is the error and under what conditions?

@danrgll
Copy link
Contributor Author

danrgll commented Jul 3, 2024

Sorry I should have highlighted this better. What I meant was that for the fidelity parameter I also can define a default and default_confidence. Is this intented?

@Neeratyoy
Copy link
Contributor

Is this intented?

Don't think so. Likely an artifact from the general SearchSpace.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
Status: No status
Development

No branches or pull requests

3 participants