Merge branch 'main' into jan/new-pr-template

BUILD.md

@@ -112,7 +112,7 @@ High-level documentation for Rerun can be found at [http://rerun.io/docs](http:/
 
 - 🌊 [C++ API docs](https://ref.rerun.io/docs/cpp) are built with `doxygen` and hosted on GitHub. Use `pixi run -e cpp cpp-docs` to build them locally. For details on the C++ doc-system, see [Writing Docs](rerun_cpp/docs/writing_docs.md).
 - 🐍 [Python API docs](https://ref.rerun.io/docs/python) are built via `mkdocs` and hosted on GitHub. For details on the Python doc-system, see [Writing Docs](rerun_py/docs/writing_docs.md).
-- 🦀 [Rust API docs](https://ref.rerun.io/docs/rust/stable) are hosted on  <https://ref.rerun.io/docs/rust/stable>. You can build them locally with: `cargo doc --all-features --no-deps --open`.
+- 🦀 [Rust API docs](https://docs.rs/rerun/) are hosted on  <https://docs.rs/rerun/>. You can build them locally with: `cargo doc --all-features --no-deps --open`.
 
 ## Building for the web
 

CHANGELOG.md

@@ -318,7 +318,7 @@ This can both greatly simplify logging code and drastically improve performance
 API documentation:
 * 🐍 [Python `send_columns` docs](https://ref.rerun.io/docs/python/stable/common/columnar_api/#rerun.send_columns)
 * 🌊 [C++ `send_columns` docs](https://ref.rerun.io/docs/cpp/stable/classrerun_1_1RecordingStream.html#ad17571d51185ce2fc2fc2f5c3070ad65)
-* 🦀 [Rust `send_columns` docs](https://ref.rerun.io/docs/rust/stable/rerun/struct.RecordingStream.html#method.send_columns)
+* 🦀 [Rust `send_columns` docs](https://docs.rs/rerun/latest/rerun/struct.RecordingStream.html#method.send_columns)
 
 API usage examples:
 <details>

README.md

@@ -73,7 +73,7 @@ You should now be able to run `rerun --help` in any terminal.
 - ⚙️ [Examples](http://rerun.io/examples)
 - 🌊 [C++ API docs](https://ref.rerun.io/docs/cpp)
 - 🐍 [Python API docs](https://ref.rerun.io/docs/python)
-- 🦀 [Rust API docs](https://ref.rerun.io/docs/rust/stable)
+- 🦀 [Rust API docs](https://docs.rs/rerun/)
 - ⁉️ [Troubleshooting](https://www.rerun.io/docs/getting-started/troubleshooting)
 
 

crates/build/re_types_builder/src/codegen/docs/mod.rs

@@ -381,9 +381,7 @@ fn list_links(is_unreleased: bool, page: &mut String, object: &Object) {
 
         putln!(
             page,
-            // TODO(#8165): docs.rs/rerun is broken right now
-            // " * 🦀 [Rust API docs for `{}`](https://docs.rs/rerun/latest/rerun/{}/{}.{}.html{speculative_marker})",
-            " * 🦀 [Rust API docs for `{}`](https://ref.rerun.io/docs/rust/stable/rerun/{}/{}.{}.html{speculative_marker})",
+            " * 🦀 [Rust API docs for `{}`](https://docs.rs/rerun/latest/rerun/{}/{}.{}.html{speculative_marker})",
             object.name,
             object.kind.plural_snake_case(),
             if object.is_struct() { "struct" } else { "enum" },

crates/build/re_types_builder/src/codegen/rust/api.rs

@@ -864,24 +864,24 @@ fn quote_trait_impls_for_datatype_or_component(
     let quoted_arrow_datatype = if let Some(forwarded_type) = forwarded_type.as_ref() {
         quote! {
             #[inline]
-            fn arrow_datatype() -> arrow2::datatypes::DataType {
-                #forwarded_type::arrow_datatype()
+            fn arrow2_datatype() -> arrow2::datatypes::DataType {
+                #forwarded_type::arrow2_datatype()
             }
         }
     } else {
         let datatype = ArrowDataTypeTokenizer(&datatype, false);
         quote! {
             #[inline]
-            fn arrow_datatype() -> arrow2::datatypes::DataType {
+            fn arrow2_datatype() -> arrow2::datatypes::DataType {
                 #![allow(clippy::wildcard_imports)]
                 use arrow2::datatypes::*;
                 #datatype
             }
         }
     };
 
-    let quoted_from_arrow = if optimize_for_buffer_slice {
-        let from_arrow_body = if let Some(forwarded_type) = forwarded_type.as_ref() {
+    let quoted_from_arrow2 = if optimize_for_buffer_slice {
+        let from_arrow2_body = if let Some(forwarded_type) = forwarded_type.as_ref() {
             let is_pod = obj
                 .try_get_attr::<String>(ATTR_RUST_DERIVE)
                 .map_or(false, |d| d.contains("bytemuck::Pod"))
@@ -890,11 +890,11 @@ fn quote_trait_impls_for_datatype_or_component(
                     .map_or(false, |d| d.contains("bytemuck::Pod"));
             if is_pod {
                 quote! {
-                    #forwarded_type::from_arrow(arrow_data).map(bytemuck::cast_vec)
+                    #forwarded_type::from_arrow2(arrow_data).map(bytemuck::cast_vec)
                 }
             } else {
                 quote! {
-                    #forwarded_type::from_arrow(arrow_data).map(|v| v.into_iter().map(Self).collect())
+                    #forwarded_type::from_arrow2(arrow_data).map(|v| v.into_iter().map(Self).collect())
                 }
             }
         } else {
@@ -923,13 +923,13 @@ fn quote_trait_impls_for_datatype_or_component(
 
         quote! {
             #[inline]
-            fn from_arrow(
+            fn from_arrow2(
                 arrow_data: &dyn arrow2::array::Array,
             ) -> DeserializationResult<Vec<Self>>
             where
                 Self: Sized
             {
-                #from_arrow_body
+                #from_arrow2_body
             }
         }
     } else {
@@ -939,7 +939,7 @@ fn quote_trait_impls_for_datatype_or_component(
     // Forward deserialization to existing datatype if it's transparent.
     let quoted_deserializer = if let Some(forwarded_type) = forwarded_type.as_ref() {
         quote! {
-            #forwarded_type::from_arrow_opt(arrow_data).map(|v| v.into_iter().map(|v| v.map(Self)).collect())
+            #forwarded_type::from_arrow2_opt(arrow_data).map(|v| v.into_iter().map(|v| v.map(Self)).collect())
         }
     } else {
         let quoted_deserializer = quote_arrow_deserializer(arrow_registry, objects, obj);
@@ -956,13 +956,13 @@ fn quote_trait_impls_for_datatype_or_component(
 
     let quoted_serializer = if let Some(forwarded_type) = forwarded_type.as_ref() {
         quote! {
-            fn to_arrow_opt<'a>(
+            fn to_arrow2_opt<'a>(
                 data: impl IntoIterator<Item = Option<impl Into<::std::borrow::Cow<'a, Self>>>>,
             ) -> SerializationResult<Box<dyn arrow2::array::Array>>
             where
                 Self: Clone + 'a,
             {
-                #forwarded_type::to_arrow_opt(data.into_iter().map(|datum| {
+                #forwarded_type::to_arrow2_opt(data.into_iter().map(|datum| {
                     datum.map(|datum| match datum.into() {
                         ::std::borrow::Cow::Borrowed(datum) => ::std::borrow::Cow::Borrowed(&datum.0),
                         ::std::borrow::Cow::Owned(datum) => ::std::borrow::Cow::Owned(datum.0),
@@ -976,7 +976,7 @@ fn quote_trait_impls_for_datatype_or_component(
 
         quote! {
             // NOTE: Don't inline this, this gets _huge_.
-            fn to_arrow_opt<'a>(
+            fn to_arrow2_opt<'a>(
                 data: impl IntoIterator<Item = Option<impl Into<::std::borrow::Cow<'a, Self>>>>,
             ) -> SerializationResult<Box<dyn arrow2::array::Array>>
             where
@@ -1015,7 +1015,7 @@ fn quote_trait_impls_for_datatype_or_component(
             #quoted_serializer
 
             // NOTE: Don't inline this, this gets _huge_.
-            fn from_arrow_opt(
+            fn from_arrow2_opt(
                 arrow_data: &dyn arrow2::array::Array,
             ) -> DeserializationResult<Vec<Option<Self>>>
             where
@@ -1024,7 +1024,7 @@ fn quote_trait_impls_for_datatype_or_component(
                 #quoted_deserializer
             }
 
-            #quoted_from_arrow
+            #quoted_from_arrow2
         }
 
         #quoted_impl_component
@@ -1173,7 +1173,7 @@ fn quote_trait_impls_for_archetype(obj: &Object) -> TokenStream {
 
                 quote! {
                     if let Some(array) = arrays_by_name.get(#field_typ_fqname_str) {
-                        <#component>::from_arrow_opt(&**array)
+                        <#component>::from_arrow2_opt(&**array)
                             .with_context(#obj_field_fqname)?
                             #quoted_collection
                     } else {
@@ -1184,7 +1184,7 @@ fn quote_trait_impls_for_archetype(obj: &Object) -> TokenStream {
                 quote! {
                     if let Some(array) = arrays_by_name.get(#field_typ_fqname_str) {
                         Some({
-                            <#component>::from_arrow_opt(&**array)
+                            <#component>::from_arrow2_opt(&**array)
                                 .with_context(#obj_field_fqname)?
                                 #quoted_collection
                         })
@@ -1199,7 +1199,7 @@ fn quote_trait_impls_for_archetype(obj: &Object) -> TokenStream {
                         .ok_or_else(DeserializationError::missing_data)
                         .with_context(#obj_field_fqname)?;
 
-                    <#component>::from_arrow_opt(&**array).with_context(#obj_field_fqname)? #quoted_collection
+                    <#component>::from_arrow2_opt(&**array).with_context(#obj_field_fqname)? #quoted_collection
                 }}
             };
 
@@ -1269,7 +1269,7 @@ fn quote_trait_impls_for_archetype(obj: &Object) -> TokenStream {
             }
 
             #[inline]
-            fn from_arrow_components(
+            fn from_arrow2_components(
                 arrow_data: impl IntoIterator<Item = (
                     ComponentName,
                     Box<dyn arrow2::array::Array>,

crates/build/re_types_builder/src/codegen/rust/arrow.rs

@@ -65,7 +65,7 @@ impl quote::ToTokens for ArrowDataTypeTokenizer<'_> {
             DataType::Extension(fqname, datatype, _metadata) => {
                 if *recursive {
                     let fqname_use = quote_fqname_as_type_path(fqname);
-                    quote!(<#fqname_use>::arrow_datatype())
+                    quote!(<#fqname_use>::arrow2_datatype())
                 } else {
                     let datatype = ArrowDataTypeTokenizer(datatype.to_logical_type(), false);
                     quote!(#datatype)

crates/build/re_types_builder/src/codegen/rust/deserializer.rs

@@ -18,9 +18,9 @@ use crate::{
 /// This short-circuits on error using the `try` (`?`) operator: the outer scope must be one that
 /// returns a `Result<_, DeserializationError>`!
 ///
-/// There is a 1:1 relationship between `quote_arrow_deserializer` and `Loggable::from_arrow_opt`:
+/// There is a 1:1 relationship between `quote_arrow_deserializer` and `Loggable::from_arrow2_opt`:
 /// ```ignore
-/// fn from_arrow_opt(data: &dyn ::arrow2::array::Array) -> DeserializationResult<Vec<Option<Self>>> {
+/// fn from_arrow2_opt(data: &dyn ::arrow2::array::Array) -> DeserializationResult<Vec<Option<Self>>> {
 ///     Ok(#quoted_deserializer)
 /// }
 /// ```
@@ -60,7 +60,7 @@ pub fn quote_arrow_deserializer(
     let data_src = format_ident!("arrow_data");
 
     let datatype = &arrow_registry.get(&obj.fqname);
-    let quoted_self_datatype = quote! { Self::arrow_datatype() };
+    let quoted_self_datatype = quote! { Self::arrow2_datatype() };
 
     let obj_fqname = obj.fqname.as_str();
     let is_enum = obj.is_enum();
@@ -494,7 +494,7 @@ enum InnerRepr {
 ///
 /// The `datatype` comes from our compile-time Arrow registry, not from the runtime payload!
 /// If the datatype happens to be a struct or union, this will merely inject a runtime call to
-/// `Loggable::from_arrow_opt` and call it a day, preventing code bloat.
+/// `Loggable::from_arrow2_opt` and call it a day, preventing code bloat.
 ///
 /// `data_src` is the runtime identifier of the variable holding the Arrow payload (`&dyn ::arrow2::array::Array`).
 /// The returned `TokenStream` always instantiates a `Vec<Option<T>>`.
@@ -517,7 +517,7 @@ fn quote_arrow_field_deserializer(
     if let DataType::Extension(fqname, _, _) = datatype {
         if objects.get(fqname).map_or(false, |obj| obj.is_enum()) {
             let fqname_use = quote_fqname_as_type_path(fqname);
-            return quote!(#fqname_use::from_arrow_opt(#data_src).with_context(#obj_field_fqname)?.into_iter());
+            return quote!(#fqname_use::from_arrow2_opt(#data_src).with_context(#obj_field_fqname)?.into_iter());
         }
     }
 
@@ -848,7 +848,7 @@ fn quote_arrow_field_deserializer(
                 unreachable!()
             };
             let fqname_use = quote_fqname_as_type_path(fqname);
-            quote!(#fqname_use::from_arrow_opt(#data_src).with_context(#obj_field_fqname)?.into_iter())
+            quote!(#fqname_use::from_arrow2_opt(#data_src).with_context(#obj_field_fqname)?.into_iter())
         }
 
         _ => unimplemented!("{datatype:#?}"),

crates/build/re_types_builder/src/codegen/rust/reflection.rs

@@ -136,7 +136,7 @@ fn generate_component_reflection(
                         || contents.contains(&format!("impl Default for super::{}", &obj.name))
                 });
         let custom_placeholder = if auto_derive_default || has_custom_default_impl {
-            quote! { Some(#type_name::default().to_arrow()?) }
+            quote! { Some(#type_name::default().to_arrow2()?) }
         } else {
             quote! { None }
         };
@@ -145,7 +145,7 @@ fn generate_component_reflection(
             ComponentReflection {
                 docstring_md: #docstring_md,
                 custom_placeholder: #custom_placeholder,
-                datatype: #type_name::arrow_datatype(),
+                datatype: #type_name::arrow2_datatype(),
             }
         };
         quoted_pairs.push(quote! { (#quoted_name, #quoted_reflection) });

crates/build/re_types_builder/src/codegen/rust/serializer.rs

@@ -19,7 +19,7 @@ pub fn quote_arrow_serializer(
 ) -> TokenStream {
     let datatype = &arrow_registry.get(&obj.fqname);
 
-    let quoted_datatype = quote! { Self::arrow_datatype() };
+    let quoted_datatype = quote! { Self::arrow2_datatype() };
 
     let is_enum = obj.is_enum();
     let is_arrow_transparent = obj.datatype.is_none();
@@ -51,7 +51,7 @@ pub fn quote_arrow_serializer(
         let bitmap_dst = format_ident!("{quoted_data_dst}_bitmap");
 
         // The choice of true or false for `elements_are_nullable` here is a bit confusing.
-        // This code-gen path forms the basis of `to_arrow_opt`, which implies that we
+        // This code-gen path forms the basis of `to_arrow2_opt`, which implies that we
         // support nullable elements. Additionally, this MAY be used as a recursive code
         // path when using an enum within a struct, and that struct within the field may
         // be null, as such the elements are always handled as nullable.
@@ -485,7 +485,7 @@ fn quote_arrow_field_serializer(
 
             return quote! {{
                 _ = #bitmap_src;
-                #fqname_use::to_arrow_opt(#data_src #option_wrapper)?
+                #fqname_use::to_arrow2_opt(#data_src #option_wrapper)?
             }};
         }
     }
@@ -903,7 +903,7 @@ fn quote_arrow_field_serializer(
 
             quote! {{
                 _ = #bitmap_src;
-                #fqname_use::to_arrow_opt(#data_src #option_wrapper)?
+                #fqname_use::to_arrow2_opt(#data_src #option_wrapper)?
             }}
         }