-
Notifications
You must be signed in to change notification settings - Fork 409
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
Implement missing code-examples for C++ #3751
Labels
Milestone
Comments
This was referenced Oct 18, 2023
Wumpf
added a commit
that referenced
this issue
Oct 18, 2023
### What * part of #3751 What it says on the tin! Tried using `sin` + cast instead of `sinf`, but still couldn't get the results to match with Rust sadly 🤷 <img width="1232" alt="image" src="https://github.com/rerun-io/rerun/assets/1220815/4bd7d025-3945-45b8-8215-5196c085d0fd"> ### Checklist * [x] I have read and agree to [Contributor Guide](https://github.com/rerun-io/rerun/blob/main/CONTRIBUTING.md) and the [Code of Conduct](https://github.com/rerun-io/rerun/blob/main/CODE_OF_CONDUCT.md) * [x] I've included a screenshot or gif (if applicable) * [x] I have tested [demo.rerun.io](https://demo.rerun.io/pr/3914) (if applicable) * [x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG - [PR Build Summary](https://build.rerun.io/pr/3914) - [Docs preview](https://rerun.io/preview/81044e7460bf76c1c3093e929e55b7a499c9f24f/docs) <!--DOCS-PREVIEW--> - [Examples preview](https://rerun.io/preview/81044e7460bf76c1c3093e929e55b7a499c9f24f/examples) <!--EXAMPLES-PREVIEW--> - [Recent benchmark results](https://ref.rerun.io/dev/bench/) - [Wasm size tracking](https://ref.rerun.io/dev/sizes/)
Wumpf
added a commit
that referenced
this issue
Oct 18, 2023
### What * part of #3751 * use Result for from_file * move heavy impl to c++ (codegen TODO: need to not leak heavy headers into public; make putting headers in _ext opt-in!) * (unrelated) rolled back some local vscode cmake settings that weren't that great after all <img width="1269" alt="image" src="https://github.com/rerun-io/rerun/assets/1220815/f98de83b-1f29-48a9-8703-39e05a9a4b26"> ### Checklist * [x] I have read and agree to [Contributor Guide](https://github.com/rerun-io/rerun/blob/main/CONTRIBUTING.md) and the [Code of Conduct](https://github.com/rerun-io/rerun/blob/main/CODE_OF_CONDUCT.md) * [x] I've included a screenshot or gif (if applicable) * [x] I have tested [demo.rerun.io](https://demo.rerun.io/pr/3913) (if applicable) * [x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG - [PR Build Summary](https://build.rerun.io/pr/3913) - [Docs preview](https://rerun.io/preview/317506389d3fc84a38d3c60db7843d35875898c2/docs) <!--DOCS-PREVIEW--> - [Examples preview](https://rerun.io/preview/317506389d3fc84a38d3c60db7843d35875898c2/examples) <!--EXAMPLES-PREVIEW--> - [Recent benchmark results](https://ref.rerun.io/dev/bench/) - [Wasm size tracking](https://ref.rerun.io/dev/sizes/)
We're pretty much done. Only relevant things left are:
Others:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
a few missing. If it's too many to do on one go put them here.
There are a few todo's scattered that may give rise to more tasks
This will also mean making sure the interfaces are good (expand our
_ext.cpp
files)The text was updated successfully, but these errors were encountered: