-
Notifications
You must be signed in to change notification settings - Fork 6
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
Understanding more about PyAPSI #15
Comments
Thanks for reaching out
There is no added functionality to the APSI protocol / the crypto. PyAPSI is just an attempt to expose APSI to a wider circle of developers and build something with it myself down the road.
A bit of the progress is reflected in my exchange with one of the APSI maintainers in #1. From an architecture perspective, the most challenging for me to wrap my head around was the notion of channels for processing the different stages. What helped me figure things out is looking at the code examples in the APSI repository, like the CLI as well as the tests.
Does looking at the contents of the Taskfile.yml already help you?
Unfortunately, I just noticed that the second one fails due to what seems to be a SEAL version mismatch. I'll try to look into getting that running again but it might take a moment (#16). In general, note that as reflected in #13 one remaining challenge with the current attempt of building wheel files is that there are hardware dependent optimizations compiled into APSI during build time that might be at odds with the target system. Even if one would be able to restrict this to the minimal subset of specialty features during compile time, this would mean that the folks installing the wheel do not benefit from all possible speed ups for APSI. An alternative approach would be to install from source and compile on the client side when PyAPSI is installed. While that would increase the compatibility and performance it would require a bit more complexity (most notably vcpkg) on the end user side. I'm not sure whether there is an obvious answer to this. |
Thanks for the guidance. I was able to completely test the CLI mode of APSI in labeled mode and it worked well (after some workarounds such as #16). I'll try to add more documentation to the main APSI repository soon. Will dig into PyAPSI more in the next few weeks and will add/ ask anything new via the issues. Thanks again! |
@LGro, great work with this wrapper. It looks well designed and possibly more user friendly than the base APSI library. I had some questions regarding your work
Thank you very much in advance.
The text was updated successfully, but these errors were encountered: