Roman Lutz
Roman Lutz
That's fair. I commented on the PR to see if @MikeyHV is still interested in continuing. Is there anything in particular we should add to the description of the issue...
I will focus on this in the coming weeks. I would like to avoid celebrating the PR's 3rd birthday next year 😆
How about using the `paths` filter in the trigger instead? That could be limited to directories that rely on it. https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#running-your-workflow-only-when-a-push-affects-specific-files
Thank you for elaborating so well on the options! I'll have a bunch of observations in random order. I'm just numbering them so that it's easier to respond. 1. Regarding...
Re 1: To clarify, I think the implementations will be separate regardless. The question is whether the user-facing API is unified via `ThresholdOptimizer` or whether we have two classes. I...
> Add CDF_DemographicParity and other relevant components to the fairlearn library. @Acornagain can you provide more detail in the description about what "other" relevant components you're adding? Similarly, the description...
@LeJit are you working on this or should we unassign it so that someone else can take it up?
Thank you, @aminadibi, for sharing your experience. I can see why that is frustrating! @MiroDudik should comment since he's one of the authors of the paper, but it sounds like...
Thank you for your contribution, @friyin ! Can you add tests similar to what we have for other classes in the same module? From a brief glance, the code looks...
> I´m still planning doing it, I will submit changes this week Awesome! Let us know if you get stuck on anything. Happy to help.