Skip to content

Conversation

@ZiluM
Copy link
Collaborator

@ZiluM ZiluM commented Nov 11, 2024

Add the Real proxy independent verification function (Hakim et al. 2016) and an example "lmr-real-pages2k-inpdt-verify.ipynb".

Only support TAS fields for verification now.

image

@ZiluM ZiluM requested a review from fzhu2e November 11, 2024 23:55
@CommonClimate
Copy link
Collaborator

This is a very useful feature! I'm curious why it hasn't been merged yet, given that there are no conflicts with the base branch and all checks have passed. It was 6 months ago...

@fzhu2e
Copy link
Owner

fzhu2e commented May 29, 2025

This is a very useful feature! I'm curious why it hasn't been merged yet, given that there are no conflicts with the base branch and all checks have passed. It was 6 months ago...

This pull request includes not only the new feature but also some unrelated modifications.
I asked @ZiluM to remove the unrelated changes before merging, but that hasn’t happened.
This was also mentioned to you on Slack 6 months ago.

@CommonClimate
Copy link
Collaborator

@ZiluM can you remove the unrelated changes so Feng can merge the relevant changes?

@ZiluM
Copy link
Collaborator Author

ZiluM commented May 30, 2025 via email

@CommonClimate
Copy link
Collaborator

Very sorry to hear of this condition @ZiluM. Please take all the time you need.

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