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

TypeError: '>' not supported between instance of 'str' and 'int' when tune_for_memory is True #2499

Closed
Jeffwan opened this issue Sep 15, 2022 · 1 comment · Fixed by #2508
Assignees

Comments

@Jeffwan
Copy link

Jeffwan commented Sep 15, 2022

Describe the bug
I tried to set tune_for_memory to True and meet TypeError.

image

To Reproduce
Steps to reproduce the behavior:

    automl_config = create_auto_config(
        dataset=dataset_df,
        target=target_column,
        time_limit_s=time_limit_s,
        tune_for_memory=tune_for_memory,
        user_config=None,
        random_seed=default_random_seed,
        use_reference_config=False,
    )


    auto_train_results = train_with_config(
        dataset=dataset_df,
        config=automl_config,
        output_directory=output_dir,
        random_seed= default_random_seed,
    )

Expected behavior
tune_for_memory should be working without any expected error

Environment (please complete the following information):

  • OS: [e.g. iOS] ludwigai/ludwig:nightly
  • Version [e.g. 22]
  • Python version 3.8
  • Ludwig version nightly
@Jeffwan Jeffwan changed the title TypeError: '>' not supported between instance of 'str' and 'int' TypeError: '>' not supported between instance of 'str' and 'int' when tune_for_memory is True Sep 15, 2022
@tgaddair
Copy link
Collaborator

Thanks @Jeffwan for pointing this out. We haven't been using tune_for_memory internally and it's experimental at the moment, but I managed to fix the issue in #2508. We should be able to land this into the 0.6 release tomorrow.

@tgaddair tgaddair self-assigned this Sep 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants