Skip to content

Added the skeleton of a C++ module to directly evaluate FTAs using llama.cpp#1

Open
tristanvdb wants to merge 51 commits intollnl:masterfrom
tristanvdb:master
Open

Added the skeleton of a C++ module to directly evaluate FTAs using llama.cpp#1
tristanvdb wants to merge 51 commits intollnl:masterfrom
tristanvdb:master

Conversation

@tristanvdb
Copy link
Contributor

No description provided.

 + using NODE(Identifier) instead of std::string
 + ensuring good source positions for all nodes
 + changing Export to Alias
 + changing PromptRef to ObjectRef
…g in unit-tests. Also some work on exception wrapper
- Add [[maybe_unused]] attributes to structured bindings in driver.cxx
  to suppress unused variable warnings when NDEBUG is defined
- Wrap exception backtrace wrapper in #ifndef NDEBUG to prevent
  linker errors in Release builds (--wrap flag only set for Debug)
- Fixed ignored-attributes warning in exception.cxx by replacing
  decltype(&pclose) with a custom PipeCloser functor struct
- Add persistent container pattern using `docker run -d ... sleep infinity`
- Document Debug vs Release builds with CMAKE_BUILD_TYPE
- Add comprehensive testing section with ctest commands
- Add Python package testing instructions
- Update container commands to use consistent /workspace path
- Add Ubuntu Docker support documentation
- Reorganize sections for better clarity

The updated documentation reflects current best practices for development
and testing using Docker containers.
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.

1 participant