Skip to content

Conversation

@victorhuangwq
Copy link
Contributor

No description provided.

@victorhuangwq victorhuangwq marked this pull request as draft November 14, 2025 23:59
@victorhuangwq
Copy link
Contributor Author

initial draft based on discussion here: #45

will update it based on relevant discussions in the thread, as well as from TPAC before marking it as ready to merge.

Copy link
Collaborator

@khushalsagar khushalsagar left a comment

Choose a reason for hiding this comment

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

Looking good!

Copy link
Contributor Author

@victorhuangwq victorhuangwq left a comment

Choose a reason for hiding this comment

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

thanks for the review. will make changes accordingly, alongside the updates that I will be making to include points from TPAC and the current issue thread.

@victorhuangwq victorhuangwq marked this pull request as ready for review November 25, 2025 00:45
@khushalsagar
Copy link
Collaborator

Left a comment for one attack vector that deserves more thinking and can be deferred to another PR. LGTM otherwise!

@victorhuangwq
Copy link
Contributor Author

@khushalsagar @bwalderman @anssiko PR is ready to be merged! Could I get either of your help to merge it?

@khushalsagar khushalsagar merged commit 0620a0b into webmachinelearning:main Dec 5, 2025
1 check passed
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.

3 participants