Skip to content

Commit bb8d839

Browse files
committed
Ellipsoids -> Ellipsoids3D
1 parent 971f04f commit bb8d839

35 files changed

+142
-142
lines changed

crates/store/re_types/definitions/rerun/archetypes.fbs

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

crates/store/re_types/definitions/rerun/archetypes/ellipsoids.fbs crates/store/re_types/definitions/rerun/archetypes/ellipsoids3d.fbs

+1-1
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ namespace rerun.archetypes;
1212
/// Opaque and transparent rendering will be supported later.
1313
///
1414
/// \example archetypes/ellipsoid_batch !api title="Batch of ellipsoids"
15-
table Ellipsoids (
15+
table Ellipsoids3D (
1616
"attr.rust.derive": "PartialEq",
1717
"attr.rust.new_pub_crate",
1818
"attr.cpp.no_field_ctors",

crates/store/re_types/src/archetypes/.gitattributes

+1-1
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

crates/store/re_types/src/archetypes/ellipsoids.rs crates/store/re_types/src/archetypes/ellipsoids3d.rs

+32-32
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

crates/store/re_types/src/archetypes/ellipsoids_ext.rs crates/store/re_types/src/archetypes/ellipsoids3d_ext.rs

+6-6
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
use crate::components::{HalfSize3D, Position3D};
22

3-
use super::Ellipsoids;
3+
use super::Ellipsoids3D;
44

5-
impl Ellipsoids {
6-
/// Creates a new [`Ellipsoids`] for spheres with the given radii.
5+
impl Ellipsoids3D {
6+
/// Creates a new [`Ellipsoids3D`] for spheres with the given radii.
77
// Note: This is not a `Radius` component because the `Radius` component is for
88
// the on-screen sizes of lines and points.
99
#[inline]
@@ -12,7 +12,7 @@ impl Ellipsoids {
1212
Self::new(radii.into_iter().map(HalfSize3D::splat))
1313
}
1414

15-
/// Creates a new [`Ellipsoids`] for spheres with the given [`Self::centers`], and
15+
/// Creates a new [`Ellipsoids3D`] for spheres with the given [`Self::centers`], and
1616
/// [`Self::half_sizes`] all equal to the given radii.
1717
// Note: This is not a `Radius` component because the `Radius` component is for
1818
// the on-screen sizes of lines and points.
@@ -25,13 +25,13 @@ impl Ellipsoids {
2525
Self::new(radii.into_iter().map(HalfSize3D::splat)).with_centers(centers)
2626
}
2727

28-
/// Creates a new [`Ellipsoids`] with [`Self::half_sizes`].
28+
/// Creates a new [`Ellipsoids3D`] with [`Self::half_sizes`].
2929
#[inline]
3030
pub fn from_half_sizes(half_sizes: impl IntoIterator<Item = impl Into<HalfSize3D>>) -> Self {
3131
Self::new(half_sizes)
3232
}
3333

34-
/// Creates a new [`Ellipsoids`] with [`Self::centers`] and [`Self::half_sizes`].
34+
/// Creates a new [`Ellipsoids3D`] with [`Self::centers`] and [`Self::half_sizes`].
3535
#[inline]
3636
pub fn from_centers_and_half_sizes(
3737
centers: impl IntoIterator<Item = impl Into<Position3D>>,

crates/store/re_types/src/archetypes/mod.rs

+3-3
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

crates/viewer/re_space_view_spatial/src/visualizers/ellipsoids.rs

+14-14
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ use re_renderer::{
44
renderer::MeshInstance, LineDrawableBuilder, PickingLayerInstanceId, RenderContext,
55
};
66
use re_types::{
7-
archetypes::Ellipsoids,
7+
archetypes::Ellipsoids3D,
88
components::{
99
ClassId, Color, FillMode, HalfSize3D, KeypointId, Position3D, Radius, Rotation3D, Text,
1010
},
@@ -32,9 +32,9 @@ use super::{
3232

3333
// ---
3434

35-
pub struct EllipsoidsVisualizer(SpatialViewVisualizerData);
35+
pub struct Ellipsoids3DVisualizer(SpatialViewVisualizerData);
3636

37-
impl Default for EllipsoidsVisualizer {
37+
impl Default for Ellipsoids3DVisualizer {
3838
fn default() -> Self {
3939
Self(SpatialViewVisualizerData::new(Some(
4040
SpatialSpaceViewKind::ThreeD,
@@ -44,7 +44,7 @@ impl Default for EllipsoidsVisualizer {
4444

4545
// NOTE: Do not put profile scopes in these methods. They are called for all entities and all
4646
// timestamps within a time range -- it's _a lot_.
47-
impl EllipsoidsVisualizer {
47+
impl Ellipsoids3DVisualizer {
4848
fn process_labels<'a>(
4949
entity_path: &'a EntityPath,
5050
centers: &'a [Position3D],
@@ -83,7 +83,7 @@ impl EllipsoidsVisualizer {
8383
mesh_instances: &mut Vec<MeshInstance>,
8484
query: &ViewQuery<'_>,
8585
ent_context: &SpatialSceneEntityContext<'_>,
86-
data: impl Iterator<Item = EllipsoidsComponentData<'a>>,
86+
data: impl Iterator<Item = Ellipsoids3DComponentData<'a>>,
8787
render_ctx: &RenderContext,
8888
) -> Result<(), SpaceViewSystemExecutionError> {
8989
let entity_path = ctx.target_entity_path;
@@ -232,7 +232,7 @@ impl EllipsoidsVisualizer {
232232

233233
// ---
234234

235-
struct EllipsoidsComponentData<'a> {
235+
struct Ellipsoids3DComponentData<'a> {
236236
// Point of views
237237
half_sizes: &'a [HalfSize3D],
238238

@@ -248,15 +248,15 @@ struct EllipsoidsComponentData<'a> {
248248
fill_mode: FillMode,
249249
}
250250

251-
impl IdentifiedViewSystem for EllipsoidsVisualizer {
251+
impl IdentifiedViewSystem for Ellipsoids3DVisualizer {
252252
fn identifier() -> re_viewer_context::ViewSystemIdentifier {
253-
"Ellipsoids".into()
253+
"Ellipsoids3D".into()
254254
}
255255
}
256256

257-
impl VisualizerSystem for EllipsoidsVisualizer {
257+
impl VisualizerSystem for Ellipsoids3DVisualizer {
258258
fn visualizer_query_info(&self) -> VisualizerQueryInfo {
259-
VisualizerQueryInfo::from_archetype::<Ellipsoids>()
259+
VisualizerQueryInfo::from_archetype::<Ellipsoids3D>()
260260
}
261261

262262
fn filter_visualizable_entities(
@@ -286,7 +286,7 @@ impl VisualizerSystem for EllipsoidsVisualizer {
286286
// Collects solid (that is, triangles rather than wireframe) instances to be drawn.
287287
let mut solid_instances: Vec<MeshInstance> = Vec::new();
288288

289-
super::entity_iterator::process_archetype::<Self, Ellipsoids, _>(
289+
super::entity_iterator::process_archetype::<Self, Ellipsoids3D, _>(
290290
ctx,
291291
view_query,
292292
context_systems,
@@ -347,7 +347,7 @@ impl VisualizerSystem for EllipsoidsVisualizer {
347347
class_ids,
348348
keypoint_ids,
349349
)| {
350-
EllipsoidsComponentData {
350+
Ellipsoids3DComponentData {
351351
half_sizes,
352352
centers: centers.unwrap_or_default(),
353353
rotations: rotations.unwrap_or_default(),
@@ -413,10 +413,10 @@ impl VisualizerSystem for EllipsoidsVisualizer {
413413
}
414414
}
415415

416-
impl TypedComponentFallbackProvider<Color> for EllipsoidsVisualizer {
416+
impl TypedComponentFallbackProvider<Color> for Ellipsoids3DVisualizer {
417417
fn fallback_for(&self, ctx: &QueryContext<'_>) -> Color {
418418
auto_color_for_entity_path(ctx.target_entity_path)
419419
}
420420
}
421421

422-
re_viewer_context::impl_component_fallback_provider!(EllipsoidsVisualizer => [Color]);
422+
re_viewer_context::impl_component_fallback_provider!(Ellipsoids3DVisualizer => [Color]);

0 commit comments

Comments
 (0)