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 basic build support #52

Open
wants to merge 19 commits into
base: main
Choose a base branch
from
Open

Conversation

tanaya-mankad
Copy link
Contributor

@tanaya-mankad tanaya-mankad commented Jul 10, 2024

This branch adds cmake build scripts so that the lattice-generated C++ code can be tested for correctness. A few items still rest on the user, as spelled out in the additions to README.md.
Lattice will now initialize a C++ source directory as a git repository, and download necessary submodules. Building the C++ project is not yet part of testing, because it still doesn't support the schema's "forward declaration" syntax.

@tanaya-mankad tanaya-mankad marked this pull request as ready for review July 12, 2024 19:36
@spahrenk
Copy link
Contributor

  • #include <template.h>' and std::unique_ptr heat_source;now appear as#include <missing-type.h>andstd::unique_ptr heat_source;`
  • Definition of initialize function for derived classes is no longer needed.
  • Statements#include <ashrae205.h> and #include <core.h> missing in some files.

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