Skip to content

Add 'proxy' to pip install dependencies - #311

Merged
yakew7 merged 1 commit into
yakew7:mainfrom
propcgamer20-png:main
Aug 24, 2026
Merged

Add 'proxy' to pip install dependencies#311
yakew7 merged 1 commit into
yakew7:mainfrom
propcgamer20-png:main

Conversation

@propcgamer20-png

Copy link
Copy Markdown
Contributor

⏸️ Paper freeze active. Do not modify anything under paper/results-frozen/, results/, the faircode/ analysis core, any audit.yaml / dataset CSV, or the reproducibility parameters (random_state, split, iteration counts, metrics) - these are frozen for a paper in peer review (see CLAUDE.md). New audits cannot merge to main yet (timing hold - they will be parked post-paper). Explainers, docs, website, and tooling are unaffected. If you think you found a bug in the analysis code, flag it in an issue - do not silently fix it.

Summary

Add 'proxy' to pip install dependencies

Type

  • Audit
  • Explainer
  • Bug fix
  • Other

Audit checklist

  • I opened or linked a corresponding issue first
  • The folder is named after the domain, not the dataset
  • unfair.py includes protected attributes and prints the required output format
  • fair.py removes protected attributes and identified proxy variables
  • Both scripts use random_state=42 and an 80/20 train/test split
  • Proxy variables were actually tested, not just guessed
  • unfair.png and fair.png are included as PNG screenshots
  • The dataset is public and accessible without login or payment
  • The dataset file is included, or DATA.md is included if the file is too large
  • README.md includes the new results row and audit section
  • A notebook was added if the audit benefits from one

Before fairness gap:

After fairness gap:

Reduction:

Protected attribute(s):

Proxy variables dropped:


Explainer checklist

  • The file is in explainers/ and uses lowercase hyphenated naming
  • It includes a plain-language definition
  • It uses a real example from this repo or a documented real-world case
  • It includes runnable Python detection or measurement code
  • It acknowledges limitations or trade-offs
  • It links to related explainers or repo projects
  • It includes 2-3 primary sources
  • The Explainers table in README.md was updated

Linked issue

Closes #298

@vercel

vercel Bot commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

@propcgamer20-png is attempting to deploy a commit to the yashkewlani2020-gmailcom's projects Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions

Copy link
Copy Markdown

@yakew7 @ahmdkaml - new PR touching a path you own, please review.

@yakew7
yakew7 merged commit 45bec82 into yakew7:main Aug 24, 2026
19 of 20 checks passed
yakew7 added a commit that referenced this pull request Aug 25, 2026
Credits AnayDhawan (#294, closing #283/#272), Shreyash0712's
follow-up (#295), new contributor VedantMadane (#312, closing #299),
propcgamer20-png (#311, closing #298), and ahmdkaml (#288, closing
#270 - manually closed since the PR never referenced the issue).
Reordered the Contributors section for AnayDhawan's new PR-count
tier. Added the 2026-W35 metrics snapshot: stars 43->47, forks
22->23, contributors 17->18 (live GitHub numbers plus VedantMadane's
first merged PR), synced across ROADMAP.md and README.md.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

audits.yml's profiler job never installs the proxy extra, so tests/test_proxy.py is silently skipped in CI

3 participants