-
Notifications
You must be signed in to change notification settings - Fork 7
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
pretty print #83
Comments
I think @danvangeest proposed a different pretty printer. I'm happy to take anything as long as it's automated. |
Unfortunately, I use our proprietary openssl provider for pretty-printing, so we can't automate that. I don't know the state of ML-KEM in the OQS provider, but I think their ML-DSA implementation isn't ready so they could have problems displaying the signature. |
Ok, I'll fashion a PR to add some pretty printing and we can change that later on if we have something non-proprietary available, or one-shot at the last minute when there aren't any changes anymore. |
Should we just use the pretty print routine from https://github.com/lamps-wg/dilithium-certificates/tree/main/examples? I have zero problem with that.
The text was updated successfully, but these errors were encountered: