Skip to content
This repository was archived by the owner on Jun 24, 2026. It is now read-only.

SOXHUB-79403: Remove model testing - #19

Merged
sarab-fairnow merged 1 commit into
mainfrom
SOX-79403_model_test_removal
Nov 24, 2025
Merged

sarab-fairnow merged 1 commit into
mainfrom
SOX-79403_model_test_removal

Conversation

@sarab-fairnow

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes model testing functionality from the FairNow API notebooks, specifically eliminating the user data bias testing capabilities as indicated by ticket SOXHUB-79403.

Key Changes:

  • Removed two complete Jupyter notebooks related to bias testing
  • Updated documentation to remove references to the removed testing workflows

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
notebooks/User Data Testing.ipynb Entire notebook deleted - removed ML model bias testing workflow using real user data
notebooks/Generating Synthetic User Bias Test Data.ipynb Entire notebook deleted - removed synthetic data generation for bias testing
notebooks/Applications API.ipynb Updated comment to remove reference to bias testing functionality
README.md Updated documentation to remove bias testing notebooks from recommended workflow and replace with Export Reports reference

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@BrianEWhipple BrianEWhipple left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit:

@sarab-fairnow
sarab-fairnow merged commit bf5dd68 into main Nov 24, 2025
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants