mmasana/FACIL

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

ashok-arjun opened this issue · 2 comments

ashok-arjun commented

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.

mkmenta commented

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).

ashok-arjun commented

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