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

Start playing with building the libpoplar wrapper in CI #37

Closed
wants to merge 234 commits into from

Conversation

giordano
Copy link
Collaborator

@giordano giordano commented Mar 13, 2024

Proof of concept for #18.

Still lots to do:

  • the resulting library is only 21 kB, it's definitely wrong, it should be over ~40 MB, investigate what's the problem (resolved: I missed to pass the input file template.cpp to the compiler, so it was building an empty library. Now I have some compilation errors, but it's good, it means it's trying to actually compile the right file)
  • fix compilation errors emerging after fixing the above point (was missing -I{includedir}/juli
  • link libjulia and libcxxwrap
  • augment platform to build different wrappers for each SDK version
  • provide method to build wrapper locally
  • remove from project of the main package redundant dependencies that are needed only to build the wrapper
  • require julia v1.9
  • write documentation

lbb and others added 30 commits September 27, 2022 02:00
Compilation of the wrapper library is moved to a `deps/build.jl` script, so that
it can be done more automatically.
This is a temporary hack until I figure out how to properly serialise the
numerical types, which have non-static properties.
Under the hood these aren't static objects that can be safely serialised, so we
need to define them as dynamic accessor methods.

Fix #1.
`Poplar.jl` is now a submodule of the package, the one providing the interface
to the Poplar SDK.
Slightly more descriptive name.  Also, index directory where the library is
built by version of julia, so that we can make multiple versions co-exists.
@giordano giordano force-pushed the mg/build-libpoplar branch 21 times, most recently from 285e742 to 8240b5f Compare March 14, 2024 12:07
@giordano
Copy link
Collaborator Author

Small update: the current state is working and all the tests (using hardware IPU) are passing when using the library compiled in CI. The proof-of-concept is definitely successful!

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.

6 participants