-
Notifications
You must be signed in to change notification settings - Fork 166
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
feat: Conversion and chain code in traccc #3656
base: main
Are you sure you want to change the base?
Conversation
A little weekend gift for @asalzburger |
cebf42a
to
ba45053
Compare
cd6a697
to
ea15a01
Compare
Quality Gate failedFailed conditions See analysis details on SonarCloud Catch issues before they fail your Quality Gate with our IDE extension SonarLint |
|
||
auto traccc = mex.def_submodule("traccc"); | ||
auto [_, mex] = ctx.get("main", "examples"); | ||
auto m = mex.def_submodule("_traccc"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does this run the code that adds a submodule multiple times?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We anyway need to streamline a bit the python bindings and make it somewhat consistent.
At this point in time, the iterator for `MeasurementContainer` does not match the `std::forward_iterator` concept which SonarCloud really wants me to use. In this commit, I make sure that the iterator satisfies the concept. Unfortunately, this requires a default-initializer, because `std::forward_iterator` requires `std::incrementable` which requires `std::regular`, which requires `std::semiregular` which in turn requires `std::default_initializable` which I think is silly.
ea15a01
to
a0c9c25
Compare
This PR adds conversion code as well as the code to run simple host-side chain algorithms using traccc. Co-authored-by: Stephen Nicholas Swatman <[email protected]>
This PR adds conversion code as well as the code to run simple host-side chain algorithms using traccc.
This code is highly experimental and perhaps not yet of the highest possible quality; I intend to iterate on this later, but I'd like to get the skeleton in to somewhat stabilize this plugin.
Closes #3281, #3320, #3363, #3505.