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

impl Copy for Arrow3D. #2239

Merged
merged 1 commit into from
May 29, 2023
Merged

impl Copy for Arrow3D. #2239

merged 1 commit into from
May 29, 2023

Conversation

kpreid
Copy link
Collaborator

@kpreid kpreid commented May 28, 2023

What

Arrow3D is a simple pair of vectors, so it should be Copy like vectors are. This will ease working with, for example, batches of arrows stored in arrays.

Checklist

PR Build Summary: {{ pr-build-summary }}

`Arrow3D` is a simple pair of vectors, so it should be `Copy` like
vectors are. This will ease working with, for example, batches of arrows
stored in arrays.
Copy link
Member

@Wumpf Wumpf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems reasonable! Thanks for the PR!

@Wumpf Wumpf added 🦀 Rust API Rust logging API 😤 annoying Something in the UI / SDK is annoying to use labels May 29, 2023
@emilk emilk merged commit c4373c6 into rerun-io:main May 29, 2023
@kpreid kpreid deleted the arrow branch May 29, 2023 17:54
emilk pushed a commit that referenced this pull request Jun 15, 2023
### What

`Arrow3D` is a simple pair of vectors, so it should be `Copy` like
vectors are. This will ease working with, for example, batches of arrows
stored in arrays.

### 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)
* [ ] ~~I've included a screenshot or gif (if applicable)~~

<!-- This line will get updated when the PR build summary job finishes.
-->
PR Build Summary: {{ pr-build-summary }}
This was referenced Jun 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
😤 annoying Something in the UI / SDK is annoying to use 🦀 Rust API Rust logging API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants