Initial code publication #1
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Horolsky
force-pushed
the
initial-code-publication
branch
3 times, most recently
from
20e16b3 to
29cdfe4
Compare
Signed-off-by: Horolsky <[email protected]>
Signed-off-by: Horolsky <[email protected]>
Signed-off-by: Horolsky <[email protected]>
Signed-off-by: Horolsky <[email protected]>
Signed-off-by: Horolsky <[email protected]>
Signed-off-by: Horolsky <[email protected]>
Horolsky
force-pushed
the
initial-code-publication
branch
from
29cdfe4 to
f6a52e2
Compare
Horolsky
force-pushed
the
initial-code-publication
branch
from
7813cf2 to
8faf320
Compare
Generate API reference with local doxygen outside of the Bazel build TODO: remove after #6
* Refine namespaces and move internals into detail ns * Remove redundant traits and metafuncs * Move precise template to decor ns * Add doxygen docs. Some trivial metafuncs omitted. * Refactor few templates Breaking: * Move json_from and dejsonize from reflect to main ns
Horolsky
force-pushed
the
initial-code-publication
branch
from
dadb352 to
73d458d
Compare
Move * packages/cxx => zmbt-framework/backends/cxx * packages/py => zmbt-framework/cli
dsemench
approved these changes
Jan 21, 2025
|
General note: |
|
@Klanor Many internal classes are in prototype state and will be refactored before MS3 with the support of static checkers. |
Klanor
reviewed
Jan 22, 2025
|
Oye, @zenseact/beltalowda, @Klanor , I've added the strategy for prototype code refinement to the pr description. |
Horolsky
force-pushed
the
initial-code-publication
branch
from
3f3ecc6 to
02e9dcc
Compare
Horolsky
force-pushed
the
initial-code-publication
branch
from
02e9dcc to
a5a897d
Compare
Add doxygen docs to Environment interfaces and update docs/environment.md. Interfaces for internal functionality left undocumented - they should be separated from the API in future.
mlisu
reviewed
Jan 22, 2025
mlisu
reviewed
Jan 22, 2025
- Fix code generation issues - Fix aliases for bitwise ops - add U+00D7 alias for cartesian
Horolsky
force-pushed
the
initial-code-publication
branch
from
679db45 to
c9c5474
Compare
mlisu
reviewed
Jan 22, 2025
Horolsky
force-pushed
the
initial-code-publication
branch
from
bcc3430 to
9c60804
Compare
Klanor
approved these changes
Jan 23, 2025
mlisu
reviewed
Jan 23, 2025
mlisu
reviewed
Jan 23, 2025
zmbt-framework/backends/cxx/include/zmbt/model/expression_api.hpp
Outdated
Show resolved
Hide resolved
mlisu
approved these changes
Jan 23, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Initial code publication
Move the source code from the internal repo, refine the structure, and add a dummy Python package.
The first commit contains the code reviewed and approved for publication by the Volvo Cars Open Source Program Office.
Most of the following changes are structural refinements or dummy configs.
A large part of implementation is in prototype state and is not a subject of review in this pr. The strategy for this debt is the following:
As one said, everything should be built top-down, except the first time 🙂