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

Backport Points2D archetype into existing Rust SDK #2387

Closed
teh-cmc opened this issue Jun 12, 2023 · 0 comments · Fixed by #2571
Closed

Backport Points2D archetype into existing Rust SDK #2387

teh-cmc opened this issue Jun 12, 2023 · 0 comments · Fixed by #2571
Assignees
Labels

Comments

@teh-cmc
Copy link
Member

teh-cmc commented Jun 12, 2023

It should be possible to start using the new object-oriented Points2D archetype from the current Rust SDK, today.

This must coexist peacefully with the existing MsgSender API.
I.e. going through the old MsgSender or through the new object API should both be legal and behave similarly.

This will allow us to start migrating step-by-step to both the object-oriented API and the codegen stack.

Requires #2383

@teh-cmc teh-cmc self-assigned this Jun 27, 2023
@teh-cmc teh-cmc linked a pull request Jun 30, 2023 that will close this issue
3 tasks
teh-cmc added a commit that referenced this issue Jul 3, 2023
**Best reviewed on a commit-by-commit basis; in particular the `rerun
codegen` commit is nothing but generated code.**

You can now use archetypes with the existing SDK:
```rust
    use rerun::experimental::archetypes::Points2D;
    MsgSender::from_archetype(
        "2d_layering/points_between_top_and_middle",
        &Points2D::new(
            (0..256).map(|i| (32.0 + (i / 16) as f32 * 16.0, 64.0 + (i % 16) as f32 * 16.0)),
        )
        .with_draw_order(1.51),
    )?
    .send(rec_stream)?;
```

Closes #2387 

---

- #2484
- #2485 
- #2487 
- #2545
- #2546
- #2549
- #2554
- #2570
- #2571

---

* [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/{{
pr.number }}) (if applicable)

- [PR Build Summary](https://build.rerun.io/pr/{{ pr.number }})
- [Docs preview](https://rerun.io/preview/{{
"pr:%s"|format(pr.branch)|encode_uri_component }}/docs)
- [Examples preview](https://rerun.io/preview/{{
"pr:%s"|format(pr.branch)|encode_uri_component }}/examples)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant