Bas

Results 19 comments of Bas

Wouldn't we do something like `Optional[Union[Sequence[Type[Exception]], Type[Exception]]]`? or equivalently `Optional[Sequence[Type[Exception]] | Type[Exception]]` This would indicate either None, a sequence or simply an exception. The default value sensibly would be None...

Similarly for states: `Optional[Sequence[optuna.trial._state.TrialState] | optuna.trial._state.TrialState]`

Ah right, good that you mention that `|` is from 3.10. Right, that makes sense for states, I'll do that then. I will work on implementing this and link you...

This will require all the classes that build on the Study class to have the type annotation changed too. These are: `optimize.py` for the lightgbm tuner `_hyperband.py` in pruners `_base.py`...

Yep, it's coming! It might be a day or two, uni is busy!

Apologies for this delay, it's my exam period in university so everything is taking a while. At the latest it'll be Saturday.

I'm going to work on implementing bridging centrality this week.

@BrandonKMLee do you mean for the other centrality measures you mentioned, Laplacian and Communicability? Or how do you mean documenting for implementing other measures?

@BrandonKMLee Have a look at the PR if you like

Is there any progress on this? Is mps usable in any way already?