Skip to content
This repository has been archived by the owner on Jul 29, 2024. It is now read-only.

Fix a little bug #21

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fix a little bug #21

wants to merge 2 commits into from

Commits on Dec 28, 2023

  1. Update eval.py, fix a minor bug 

    Dividing by an additional N does not affect the trend of the curve, but it makes the Adjusted Qini and Cgain curves very similar on data where the control and treatment groups are relatively evenly distributed (after sorted by uplift value), making the two curves difficult to distinguish and prone to misunderstanding.
    @Fanshaoliu
    Fanshaoliu committed Dec 28, 2023
    Configuration menu
    Copy the full SHA
    1b93b75 View commit details
    Browse the repository at this point in the history

  2. Merge pull request #1 from Fanshaoliu/Fanshaoliu 

    Update eval.py, fix a minor bug
    @Fanshaoliu
    Fanshaoliu committed Dec 28, 2023
    Configuration menu
    Copy the full SHA
    fe5c07d View commit details
    Browse the repository at this point in the history