Roman Lutz

Results 458 comments of Roman Lutz

@hildeweerts it’s definitely relevant. @adrinjalali suggested using GridSearchCV for this a while back, so if we make such a change we could get that perhaps for free.

The consensus in #517 was not to use `tqdm` to avoid introducing a new dependency. Instead, `logging` is preferable as @adrinjalali explained in #517 . If anyone wants to start...

Hi @excepto64 ! That's correct. We wrote the fetch function so that you can query by state https://fairlearn.org/v0.10/api_reference/generated/fairlearn.datasets.fetch_acs_income.html#fairlearn.datasets.fetch_acs_income Does that help?

> > * If you don't know what I mean please lmk and I'll point you the right way. > > Hello Roman, > > If you don’t mind, could...

> Hello @rlundeen2 & @romanlutz , > > When running `pre-commit run --all`, I encountered a MyPy type-checking error in `doc/code/orchestrators/3_xpia_orchestrator.py` at line 192. The issue is an **incompatible type...

Just wanted to say we haven't forgotten. I'm working through a list of things right now, but as soon as I have a couple of hours I'll give this a...

Question: do you think it makes sense to keep it as is for this PR? I'm hoping to test it next week and then hopefully merge. If we make modifications...

I finally managed to run it myself. It was surprisingly easy, or in other words: great job! We need a little more explanation in the notebook where it says "RAG...