FACIL icon indicating copy to clipboard operation
FACIL copied to clipboard

gridsearch-tasks == -1 in args does not perform grid search

Open ashok-arjun opened this issue 2 years ago • 2 comments

Hello!

Thank you very much for this library - it's great!

I have a question:

The default value of --gridsearch-tasks in main_incremental.py is -1. And it has been given that setting it to -1 will perform grid search for all tasks.

But this does not reflect in the code, and grid search is not done when this is executed.

The only lines where gridsearch_tasks is referenced are line 167, line 214 and line 241 and all of these work only when the value is positive.

Can you please check if the code is right?

Thank you.

ashok-arjun avatar Aug 07 '21 17:08 ashok-arjun

Hi, thanks for the issue!

I think you are right! I will check it out.

Thank you so much!

(Noting, just in case: setting it to the value of --num-tasks as done here should run the full GridSearch).

mkmenta avatar Aug 08 '21 21:08 mkmenta

Thank you @mkmenta. Got it, will use --num-tasks.

ashok-arjun avatar Aug 10 '21 01:08 ashok-arjun