Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gpj inference risk each against rest cols #74

Merged
merged 4 commits into from
May 29, 2024

Conversation

gpadres
Copy link
Collaborator

@gpadres gpadres commented May 23, 2024

Description

Adding inference risk evaluation for each column as secret with rest of columns as auxiliary information

How Has This Been Tested?

Tests included

@gpadres gpadres added the enhancement New feature or request label May 23, 2024
@gpadres gpadres requested a review from johanos1 May 23, 2024 09:57
Copy link

github-actions bot commented May 23, 2024

Tests Skipped Failures Errors Time
133 0 💤 0 ❌ 0 🔥 29.269s ⏱️

Copy link
Collaborator

@johanos1 johanos1 left a comment

Choose a reason for hiding this comment

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

Looks good. I think the code is quite neat. I will approve this PR as soon as the other PR is approved.

@johanos1 johanos1 self-requested a review May 29, 2024 12:42
johanos1
johanos1 previously approved these changes May 29, 2024
@gpadres gpadres dismissed johanos1’s stale review May 29, 2024 12:42

The merge-base changed after approval.

johanos1
johanos1 previously approved these changes May 29, 2024
@gpadres gpadres dismissed johanos1’s stale review May 29, 2024 13:03

The merge-base changed after approval.

@gpadres gpadres merged commit ed4d088 into main May 29, 2024
2 checks passed
@gpadres gpadres deleted the gpj_inference_risk_each_against_rest_cols branch May 29, 2024 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants