pca fix progress bar

[Moo-Marc]

Improved progress bar "nesting" for recent modifications (PCA, flattening, scouts)

[Edouard2laire]

This is interesting. I am wondering if instead of 'nesting' progress bar, if we should not instead allow to have multiple progress bars (#614)

Maybe something like this (https://www.mathworks.com/matlabcentral/fileexchange/26589-multiwaitbar):

[Moo-Marc]

Thanks @rcassani, I'll make that a new bst_progress command: pBar = bst_progress('reset', pBar)
Would you have another suggestion instead of calling it 'reset'? Maybe 'restart', or even overload 'start'?

Hi @Edouard2laire, indeed I saw your post at the time and like it, but I doubt we'll have resources for such improvements soon, with François being away. This PR is just a quick fix to follow Bst "nesting conventions" - my recent changes had made some processes stick at 100% (e.g. connectivity with PCA scouts).

[rcassani]

Thanks @Edouard2laire, indeed that PR is in the queue of things to do.

Would you have another suggestion instead of calling it 'reset'? Maybe 'restart', or even overload 'start'?

I think we can add two methods 'GetBar' and 'SetBar'.
Will do

oldBar = bst_progress('GetBar');
bst_progress('SetBar', oldBar);

[rcassani]

@Moo-Marc, last two commits should do the trick. Can you test that it behaves as expected?

[Moo-Marc]

@rcassani Few additional tweaks. Works ok now!