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.

Sign up for GitHub

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

Jump to bottom

Running flaml[tune] using "-O" flag for python interpreter (optimization - disables assertions) crashes #981

Open
bbudescu opened this issue Apr 7, 2023 · 3 comments · May be fixed by #1169
Open

Running flaml[tune] using "-O" flag for python interpreter (optimization - disables assertions) crashes #981

bbudescu opened this issue Apr 7, 2023 · 3 comments · May be fixed by #1169
Assignees
@luckyklyist
Labels
good first issue Good for newcomers

Comments

@bbudescu
Copy link

bbudescu commented Apr 7, 2023

this is the stack trace:

analysis = tune.run(
  File "/home/ec2-user/pyvenv/lib64/python3.9/site-packages/flaml/tune/tune.py", line 536, in run
    search_alg = SearchAlgorithm(
  File "/home/ec2-user/pyvenv/lib64/python3.9/site-packages/flaml/tune/searcher/blendsearch.py", line 248, in __init__
    points_to_evaluate=self._evaluated_points,
AttributeError: 'BlendSearch' object has no attribute '_evaluated_points'
@sonichi
Copy link
Collaborator

sonichi commented Apr 7, 2023

It's easy to fix by modifying this line:

FLAML/flaml/tune/searcher/blendsearch.py

Line 241 in 7f9402b

assert evaluated_rewards

Please feel free to create a PR.

@sonichi sonichi added the good first issue Good for newcomers label Apr 7, 2023
@luckyklyist
Copy link
Contributor

is anyone working on this ?if none then can assign this issue to me @sonichi

@sonichi
Copy link
Collaborator

sonichi commented Apr 26, 2023

is anyone working on this ?if none then can assign this issue to me @sonichi

Sure. Assigned to you.

@bbudescu bbudescu linked a pull request Aug 3, 2023 that will close this issue
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@luckyklyist luckyklyist

Labels
good first issue Good for newcomers
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Enable running FLAML using the -O flag for the python interpreter bbudescu/FLAML
3 participants
@bbudescu @sonichi @luckyklyist