Add additional parameters to generate_counterfactuals() function

[gaugup]

It seems the generate_counterfactuals() method in dice_genetic.py support many more arguments than the ones supported in generate_counterfactuals() in explainer_base.py. Adding the additional parameters to generate_counterfactuals() in explainer_base.py.

Signed-off-by: gaugup gaugup@microsoft.com

[github-actions]

Unit Test Results

    1 files  ±0      1 suites  ±0   12m 51s ⏱️ -11s
221 tests ±0  216 ✔️ ±0  5 💤 ±0  0 ❌ ±0 

Results for commit 9c4d58b. ± Comparison against base commit e6a1dc3.

♻️ This comment has been updated with latest results.

[amit-sharma]

I like the idea of making it consistent. But the problem is that all methods do not support these parameters. For example, dice_random will ignore categorical_penalty, even if a user sets its value.

For each parameter, shall we write in the docstring on which algorithms support it? Would that be a good way to communicate this? If that sounds good, I'm happy to edit the docstring with this info.

[gaugup]

I like the idea of making it consistent. But the problem is that all methods do not support these parameters. For example, dice_random will ignore categorical_penalty, even if a user sets its value.

For each parameter, shall we write in the docstring on which algorithms support it? Would that be a good way to communicate this? If that sounds good, I'm happy to edit the docstring with this info.

Sure, please go ahead and add the relevant docstrings.

[amit-sharma]

Okay will add these in the next few days.