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

Add WF1 #15

Merged
merged 9 commits into from
Jun 6, 2024
Merged

Add WF1 #15

merged 9 commits into from
Jun 6, 2024

Conversation

nicelhc13
Copy link
Contributor

Description

This PR ports WF1 to pando-mono.
Note that link prediction was removed due to formatting issues. If you need this, I will revert it back.
WF1's tests are not added since they were broken while the repo was being updated.

Checklist

  • The additions follow the code standards in the developer guide.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@nicelhc13
Copy link
Contributor Author

Other than porting to the new Galois interface, this should be ready to be merged (in terms of code migration)

@AdityaAtulTewari
Copy link
Contributor

What was the command that was failing?

@nicelhc13
Copy link
Contributor Author

it runs on 2 pxns, data.00001.csv, and its accuracy was correctly increased to 45% from 33%. This requires 20K L1SP memory. I think this is ready to merge.

@nicelhc13 nicelhc13 merged commit 003a4ae into main Jun 6, 2024
20 checks passed
@nicelhc13 nicelhc13 deleted the users/hochan/port-wf1 branch June 6, 2024 07:43
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