Skip to content

Fix Distribution Inference Module#21

Open
asim29 wants to merge 3 commits intomainfrom
fix-distribution-inference
Open

Fix Distribution Inference Module#21
asim29 wants to merge 3 commits intomainfrom
fix-distribution-inference

Conversation

@asim29
Copy link
Collaborator

@asim29 asim29 commented Jul 22, 2024

A lot of work needs to be done in this module, specifically on a high-level:

  • Need to design a strategy such that the attack module itself works on a single target model. However, the evaluation requires multiple modules. The metrics module might need to implement a full pipeline.
  • Need to generalize the code such that it works with any number of sensitive attributes, using only their indexes rather than hardcoded values for the sensitive attributes.

@asim29 asim29 requested a review from vasishtduddu July 22, 2024 19:56
Copy link
Contributor

@sebszyller sebszyller left a comment

Choose a reason for hiding this comment

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

types missing in the attack

@asim29
Copy link
Collaborator Author

asim29 commented Jul 26, 2024

I added types to a lot of the extra functions, but the main attack function (SuriSATML2023.attack()) I will likely change entirely. I'll do that in the same PR.

@sebszyller
Copy link
Contributor

this is still WIP, right?

@asim29
Copy link
Collaborator Author

asim29 commented Sep 16, 2024

this is still WIP, right?

yes, slightly lower priority right now

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.

2 participants