enabled commandline with number of indices as argument

[PaulJonasJost]

So i think this should work as a first draft, i would need to enable parsing comma separated ids or indices as well, which i will try to finish this week. Also i noticed that the option -t should basically only be used when we want to perform a gradient check? I would suggest removing -t and going for -g for the gradient check. But could also be that i missed something.

[dweindl]

i would need to enable parsing comma separated ids or indices as well, which i will try to finish this week.

👍

Also i noticed that the option -t should basically only be used when we want to perform a gradient check? I would suggest removing -t and going for -g for the gradient check. But could also be that i missed something.

The default is optimization, you can specify that explicitly or not. It was meant to be extendable to future tasks that might be added. No strong opinion. Command line interface could take some improvements anyways. If you change it this way, this also needs to be updated in some tests and examples.

You seem to have overwritten a couple of my previous changes. Please check the diff and revert the probably unintended changes.

[paulstapor]

What's the status here? Can be merged? One more round of review?

[paulstapor]

Looks good to me... At a second glance: not sure whether things also work if no parameter indices are passed... But this should fix the current problem for the moment

[dweindl]

Please add a test case similar to

parPE/examples/parpeamici/steadystate/test_steadystate.py

Line 35 in ee1de80

def test_nompi_gradient_check():

[sonarcloud]

SonarCloud Quality Gate failed.

0 Bugs
0 Vulnerabilities
0 Security Hotspots
4 Code Smells

21.4% Coverage
0.0% Duplication