chore: add cargo shear exception for getrandom

crates/ascend/Cargo.toml

@@ -9,7 +9,7 @@ publish = false
 crate-type = ["cdylib", "rlib"]
 
 [dependencies]
-axum = { version = "0.7", optional = true }
+axum = { version = "0.8", optional = true }
 camino = { version = "1.1", optional = true }
 chrono = { version = "0.4.39", features = ["now", "serde"] }
 clap = { version = "4.5.7", features = ["derive"] }
@@ -23,18 +23,17 @@ derive_more = { version = "2", features = [
 ] }
 http = "1"
 image = { version = "0.25", optional = true }
-leptos = { version = "0.7.7", features = ["tracing"] }
-leptos_axum = { version = "0.7", optional = true }
-leptos_meta = { version = "0.7" }
-leptos_router = { version = "0.7.0" }
+leptos = { version = "0.8", features = ["tracing"] }
+leptos_axum = { version = "0.8", optional = true }
+leptos_meta = { version = "0.8" }
+leptos_router = { version = "0.8" }
 moonboard-parser = { workspace = true, optional = true }
 rand = { version = "0.9", default-features = false, features = ["thread_rng"] }
 ron = { version = "0.8" }
 serde = { version = "1", features = ["derive"] }
-server_fn = { version = "0.7.4", features = ["cbor"] }
+server_fn = { version = "0.8", features = ["cbor"] }
 smart-default = "0.7.1"
 tokio = { version = "1", features = ["rt-multi-thread"], optional = true }
-tower = { version = "0.4", optional = true }
 tower-http = { version = "0.5", features = ["fs"], optional = true }
 tracing = { version = "0.1" }
 tracing-subscriber = { version = "0.3.18", features = ["env-filter"] }
@@ -46,16 +45,12 @@ xdg = { version = "2.5", optional = true }
 uuid = { version = "1.12", features = ["serde", "v4"] }
 redb = { version = "2.4", optional = true }
 bincode = { version = "1.3", optional = true }
-serde_json = { version = "1" }
 codee = { version = "0.3" }
 error_reporter = { version = "1" }
 getrandom = { version = "0.3.1" }
 
-[dev-dependencies.serde_json]
-version = "1"
-
-[dev-dependencies.test-try]
-version = "0.1"
+[dev-dependencies]
+test-try = "0.1"
 
 [features]
 hydrate = ["leptos/hydrate", "getrandom/wasm_js", "uuid/js"]
@@ -65,7 +60,6 @@ ssr = [
   "dep:image",
   "dep:bincode",
   "dep:tokio",
-  "dep:tower",
   "dep:tower-http",
   "dep:leptos_axum",
   "dep:confik",
@@ -77,6 +71,11 @@ ssr = [
   "leptos_router/ssr",
 ]
 
+
+[package.metadata.cargo-shear]
+# getrandom is depended on in order to turn on its wasm feature indirectly
+ignored = ["getrandom"]
+
 [package.metadata.leptos]
 # The name used by wasm-bindgen/cargo-leptos for the JS/WASM bundle. Defaults to the crate name
 output-name = "ascend"

crates/ascend/src/app.rs

@@ -43,8 +43,7 @@ pub fn App() -> impl IntoView {
             <Routes fallback=|| "Not found">
                 <Route path=path!("/") view=Home />
                 <Route path=path!("/wall/:wall_uid") view=pages::wall::Page />
-                <Route path=path!("/wall/:wall_uid/edit") view=pages::edit_wall::EditWall />
-                <Route path=path!("/wall/:wall_uid/routes") view=pages::routes::Routes />
+                <Route path=path!("/wall/:wall_uid/holds") view=pages::holds::Page />
             </Routes>
         </Router>
     }

crates/ascend/src/codec.rs

@@ -3,19 +3,22 @@ pub mod ron {
 
     use codee::Decoder;
     use codee::Encoder;
+    use leptos::prelude::FromServerFnError;
+    use leptos::prelude::ServerFnErrorErr;
     use serde::Deserialize;
     use serde::Serialize;
     use serde::de::DeserializeOwned;
-    use server_fn::ServerFnError;
+    use server_fn::ContentType;
     use server_fn::codec::Encoding;
     use server_fn::codec::FromReq;
     use server_fn::codec::FromRes;
     use server_fn::codec::IntoReq;
     use server_fn::codec::IntoRes;
+    use server_fn::error::IntoAppError;
     use server_fn::request::ClientReq;
     use server_fn::request::Req;
     use server_fn::response::ClientRes;
-    use server_fn::response::Res;
+    use server_fn::response::TryRes;
 
     pub struct Ron;
 
@@ -44,10 +47,13 @@ pub mod ron {
     }
 
     impl Encoding for Ron {
-        const CONTENT_TYPE: &'static str = "application/ron";
         const METHOD: http::Method = http::Method::POST;
     }
 
+    impl ContentType for Ron {
+        const CONTENT_TYPE: &'static str = "application/ron";
+    }
+
     #[derive(Debug, Clone)]
     pub struct RonEncoded<T>(pub T);
 
@@ -74,9 +80,10 @@ pub mod ron {
     where
         Request: ClientReq<Err>,
         T: Serialize,
+        Err: FromServerFnError,
     {
-        fn into_req(self, path: &str, accepts: &str) -> Result<Request, ServerFnError<Err>> {
-            let data = Ron::encode(&self.0).map_err(|e| ServerFnError::Serialization(e.to_string()))?;
+        fn into_req(self, path: &str, accepts: &str) -> Result<Request, Err> {
+            let data = Ron::encode(&self.0).map_err(|e| ServerFnErrorErr::Serialization(e.to_string()).into_app_error())?;
             Request::try_new_post(path, Ron::CONTENT_TYPE, accepts, data)
         }
     }
@@ -86,21 +93,25 @@ pub mod ron {
     where
         Request: Req<Err> + Send,
         T: DeserializeOwned,
+        Err: FromServerFnError,
     {
-        async fn from_req(req: Request) -> Result<Self, ServerFnError<Err>> {
+        async fn from_req(req: Request) -> Result<Self, Err> {
             let data = req.try_into_string().await?;
-            Ron::decode(&data).map(RonEncoded).map_err(|e| ServerFnError::Args(e.to_string()))
+            Ron::decode(&data)
+                .map(RonEncoded)
+                .map_err(|e| ServerFnErrorErr::Args(e.to_string()).into_app_error())
         }
     }
 
     // IntoRes
-    impl<CustErr, T, Response> IntoRes<Ron, Response, CustErr> for RonEncoded<T>
+    impl<Err, T, Response> IntoRes<Ron, Response, Err> for RonEncoded<T>
     where
-        Response: Res<CustErr>,
+        Response: TryRes<Err>,
         T: Serialize + Send,
+        Err: FromServerFnError,
     {
-        async fn into_res(self) -> Result<Response, ServerFnError<CustErr>> {
-            let data = Ron::encode(&self.0).map_err(|e| ServerFnError::Serialization(e.to_string()))?;
+        async fn into_res(self) -> Result<Response, Err> {
+            let data = Ron::encode(&self.0).map_err(|e| ServerFnErrorErr::Serialization(e.to_string()).into_app_error())?;
             Response::try_from_string(Ron::CONTENT_TYPE, data)
         }
     }
@@ -110,12 +121,13 @@ pub mod ron {
     where
         Response: ClientRes<Err> + Send,
         T: DeserializeOwned,
+        Err: FromServerFnError,
     {
-        async fn from_res(res: Response) -> Result<Self, ServerFnError<Err>> {
+        async fn from_res(res: Response) -> Result<Self, Err> {
             let data = res.try_into_string().await?;
             Ron::decode(&data)
                 .map(RonEncoded)
-                .map_err(|e| ServerFnError::Deserialization(e.to_string()))
+                .map_err(|e| ServerFnErrorErr::Deserialization(e.to_string()).into_app_error())
         }
     }
 

crates/ascend/src/components.rs

@@ -19,9 +19,9 @@ use leptos::prelude::*;
 #[component]
 pub fn OnHoverRed(children: Children) -> impl IntoView {
     view! {
-        <div class="group relative">
+        <div class="relative group">
             <div>{children()}</div>
-            <div class="absolute inset-0 bg-red-500 opacity-0 group-hover:opacity-50"></div>
+            <div class="absolute inset-0 bg-red-500 opacity-0 group-hover:opacity-50" />
         </div>
     }
 }

crates/ascend/src/components/attempt.rs

@@ -7,7 +7,7 @@ use leptos::prelude::*;
 #[component]
 #[tracing::instrument(skip_all)]
 pub fn Attempt(#[prop(into)] date: Signal<DateTime<Utc>>, #[prop(into)] attempt: Signal<Option<models::Attempt>>) -> impl IntoView {
-    tracing::trace!("Enter");
+    crate::tracing::on_enter!();
 
     let s = time_ago(date.get());
 
@@ -19,9 +19,7 @@ pub fn Attempt(#[prop(into)] date: Signal<DateTime<Utc>>, #[prop(into)] attempt:
     };
 
     let text_color = match attempt.get() {
-        Some(models::Attempt::Flash) => "text-cyan-500",
-        Some(models::Attempt::Send) => "text-teal-500",
-        Some(models::Attempt::Attempt) => "text-pink-500",
+        Some(attempt) => attempt.gradient().class_text(),
         None => "",
     };
 

crates/ascend/src/components/button.rs

@@ -19,34 +19,40 @@ pub fn Button(
 ) -> impl IntoView {
     let margin = "mx-2 my-1 sm:mx-5 sm:my-2.5";
 
-    let icon_view = icon.get().map(|i| {
-        let icon_view = i.into_view();
-        let mut classes = "self-center".to_string();
-        classes.push(' ');
-        classes.push_str(margin);
-        classes.push(' ');
-        classes.push_str(color.class_text());
+    let icon_view = move || {
+        icon.get().map(|i| {
+            let icon_view = i.into_view();
+            let mut classes = "self-center".to_string();
+            classes.push(' ');
+            classes.push_str(margin);
+            classes.push(' ');
+            classes.push_str(color.class_text());
 
-        view! { <div class=classes>{icon_view}</div> }
-    });
+            view! { <div class=classes>{icon_view}</div> }
+        })
+    };
 
-    let separator = (icon.read().is_some() && text.read().is_some()).then(|| {
-        let mut classes = "w-0.5 bg-linear-to-br min-w-0.5".to_string();
-        classes.push(' ');
-        classes.push_str(color.class_from());
-        classes.push(' ');
-        classes.push_str(color.class_to());
+    let separator = move || {
+        (icon.read().is_some() && text.read().is_some()).then(|| {
+            let mut classes = "w-0.5 bg-linear-to-br min-w-0.5".to_string();
+            classes.push(' ');
+            classes.push_str(color.class_from());
+            classes.push(' ');
+            classes.push_str(color.class_to());
 
-        view! { <div class=classes /> }
-    });
+            view! { <div class=classes /> }
+        })
+    };
 
-    let text_view = text.get().map(|text| {
-        let mut classes = "self-center uppercase w-full text-sm sm:text-base md:text-lg font-thin".to_string();
-        classes.push(' ');
-        classes.push_str(margin);
+    let text_view = move || {
+        text.get().map(|text| {
+            let mut classes = "self-center uppercase w-full text-sm sm:text-base md:text-lg font-thin".to_string();
+            classes.push(' ');
+            classes.push_str(margin);
 
-        view! { <div class=classes>{text}</div> }
-    });
+            view! { <div class=classes>{text}</div> }
+        })
+    };
 
     let class = move || {
         let mut classes = vec![];

crates/ascend/src/components/checkbox.rs

@@ -8,7 +8,7 @@ pub fn Checkbox(checked: RwSignal<bool>, #[prop(into)] text: Signal<String>, #[p
     let unique_id = Oco::from(format!("checkbox-{}", uuid::Uuid::new_v4()));
 
     let checkbox_view = view! {
-        <div class="self-center text-white bg-white rounded-xs aspect-square mx-5 my-2.5">
+        <div class="self-center my-2.5 mx-5 text-white bg-white rounded-xs aspect-square">
             <span class=("text-gray-950", move || checked.get())>
                 <icons::Check />
             </span>
@@ -25,7 +25,7 @@ pub fn Checkbox(checked: RwSignal<bool>, #[prop(into)] text: Signal<String>, #[p
     };
 
     let text_view = view! {
-        <div class="self-center mx-5 my-2.5 uppercase w-full text-lg font-thin">
+        <div class="self-center my-2.5 mx-5 w-full text-lg font-thin uppercase">
             {move || text.get()}
         </div>
     };

crates/ascend/src/components/icons.rs

@@ -7,6 +7,7 @@ pub enum Icon {
     WrenchSolid,
     ForwardSolid,
     Check,
+    Heart,
     HeartOutline,
     ArrowPath,
     PaperAirplaneSolid,
@@ -26,6 +27,7 @@ impl Icon {
             Icon::WrenchSolid => view! { <WrenchSolid /> }.into_any(),
             Icon::ForwardSolid => view! { <ForwardSolid /> }.into_any(),
             Icon::Check => view! { <Check /> }.into_any(),
+            Icon::Heart => view! { <Heart /> }.into_any(),
             Icon::HeartOutline => view! { <HeartOutline /> }.into_any(),
             Icon::ArrowPath => view! { <ArrowPath /> }.into_any(),
             Icon::PaperAirplaneSolid => view! { <PaperAirplaneSolid /> }.into_any(),
@@ -123,6 +125,20 @@ pub fn Check() -> impl IntoView {
     }
 }
 
+#[component]
+pub fn Heart() -> impl IntoView {
+    view! {
+        <svg
+            xmlns="http://www.w3.org/2000/svg"
+            viewBox="0 0 24 24"
+            fill="currentColor"
+            class="size-6"
+        >
+            <path d="m11.645 20.91-.007-.003-.022-.012a15.247 15.247 0 0 1-.383-.218 25.18 25.18 0 0 1-4.244-3.17C4.688 15.36 2.25 12.174 2.25 8.25 2.25 5.322 4.714 3 7.688 3A5.5 5.5 0 0 1 12 5.052 5.5 5.5 0 0 1 16.313 3c2.973 0 5.437 2.322 5.437 5.25 0 3.925-2.438 7.111-4.739 9.256a25.175 25.175 0 0 1-4.244 3.17 15.247 15.247 0 0 1-.383.219l-.022.012-.007.004-.003.001a.752.752 0 0 1-.704 0l-.003-.001Z" />
+        </svg>
+    }
+}
+
 #[component]
 pub fn HeartOutline() -> impl IntoView {
     view! {

crates/ascend/src/components/outlined_box.rs

@@ -22,12 +22,14 @@ pub fn OutlinedBox(children: Children, color: Gradient, #[prop(optional)] highli
         let mut c = "py-1.5 rounded-md".to_string();
         if highlight() {
             let bg = match color {
-                Gradient::PinkOrange => "bg-pink-900",
+                Gradient::PinkOrange => "bg-rose-900",
                 Gradient::CyanBlue => "bg-cyan-800",
-                Gradient::TealLime => "bg-teal-700",
-                Gradient::PurplePink => "bg-purple-900",
+                Gradient::TealLime => "bg-emerald-700",
+                Gradient::PurplePink => "bg-fuchsia-950",
                 Gradient::PurpleBlue => "bg-purple-900",
                 Gradient::Orange => "bg-orange-900",
+                Gradient::Pink => "bg-pink-900",
+                Gradient::PinkRed => "bg-fuchsia-950",
             };
 
             c.push(' ');

crates/ascend/src/components/problem.rs

@@ -19,7 +19,7 @@ pub fn Problem(
         for row in 0..dim.get().rows {
             for col in 0..dim.get().cols {
                 let hold_position = models::HoldPosition { row, col };
-                let role = move || problem.get().holds.get(&hold_position).copied();
+                let role = move || problem.read().pattern.pattern.get(&hold_position).copied();
                 let role = Signal::derive(role);
                 let hold = view! { <Hold role /> };
                 holds.push(hold);

crates/ascend/src/components/problem_info.rs

@@ -6,15 +6,15 @@ use leptos::prelude::*;
 pub fn ProblemInfo(#[prop(into)] problem: Signal<models::Problem>) -> impl IntoView {
     tracing::trace!("Enter problem info");
 
-    let name = Signal::derive(move || problem.read().name.clone());
-    let set_by = Signal::derive(move || problem.read().set_by.clone());
     let method = Signal::derive(move || problem.read().method.to_string());
+    // let name = Signal::derive(move || problem.read().name.clone());
+    // let set_by = Signal::derive(move || problem.read().set_by.clone());
 
     view! {
-        <div class="grid grid-rows-none gap-y-1 gap-x-0.5 grid-cols-[auto_1fr]">
-            <NameValue name="Name:" value=name />
+        <div class="grid grid-rows-none gap-x-0.5 gap-y-1 grid-cols-[auto_1fr]">
             <NameValue name="Method:" value=method />
-            <NameValue name="Set By:" value=set_by />
+        // <NameValue name="Name:" value=name />
+        // <NameValue name="Set By:" value=set_by />
         </div>
     }
 }
@@ -23,7 +23,7 @@ pub fn ProblemInfo(#[prop(into)] problem: Signal<models::Problem>) -> impl IntoV
 #[tracing::instrument(skip_all)]
 fn NameValue(#[prop(into)] name: Signal<String>, #[prop(into)] value: Signal<String>) -> impl IntoView {
     view! {
-        <p class="font-light mr-4 text-right text-orange-300">{name.get()}</p>
+        <p class="mr-4 font-light text-right text-orange-300">{name.get()}</p>
         <p class="text-white">{value.get()}</p>
     }
 }

crates/ascend/src/gradient.rs

@@ -7,9 +7,11 @@ pub enum Gradient {
     PurplePink,
     #[default]
     Orange,
+    Pink,
+    PinkRed,
 }
 impl Gradient {
-    pub fn class_from(&self) -> &str {
+    pub fn class_from(&self) -> &'static str {
         match self {
             Gradient::PinkOrange => "from-pink-500",
             Gradient::CyanBlue => "from-cyan-500",
@@ -17,10 +19,12 @@ impl Gradient {
             Gradient::PurplePink => "from-purple-500",
             Gradient::PurpleBlue => "from-purple-600",
             Gradient::Orange => "from-orange-400",
+            Gradient::Pink => "from-pink-400",
+            Gradient::PinkRed => "from-pink-400",
         }
     }
 
-    pub fn class_to(&self) -> &str {
+    pub fn class_to(&self) -> &'static str {
         match self {
             Gradient::PinkOrange => "to-orange-400",
             Gradient::CyanBlue => "to-blue-500",
@@ -28,17 +32,21 @@ impl Gradient {
             Gradient::PurplePink => "to-pink-500",
             Gradient::PurpleBlue => "to-blue-500",
             Gradient::Orange => "to-orange-500",
+            Gradient::Pink => "to-pink-500",
+            Gradient::PinkRed => "to-red-500",
         }
     }
 
-    pub fn class_text(&self) -> &str {
+    pub fn class_text(&self) -> &'static str {
         match self {
-            Gradient::PinkOrange => "text-pink-500",
+            Gradient::PinkOrange => "text-rose-400",
             Gradient::CyanBlue => "text-cyan-500",
-            Gradient::TealLime => "text-teal-300",
-            Gradient::PurplePink => "text-purple-500",
+            Gradient::TealLime => "text-emerald-300",
+            Gradient::PurplePink => "text-fuchsia-500",
             Gradient::PurpleBlue => "text-purple-600",
             Gradient::Orange => "text-orange-400",
+            Gradient::Pink => "text-pink-400",
+            Gradient::PinkRed => "text-pink-400",
         }
     }
 }

crates/ascend/src/models.rs

@@ -8,22 +8,68 @@ pub use v2::ImageFilename;
 pub use v2::ImageResolution;
 pub use v2::ImageUid;
 pub use v2::Method;
-pub use v2::Problem;
-pub use v2::ProblemUid;
 pub use v2::Root;
-pub use v2::Wall;
 pub use v2::WallDimensions;
 pub use v2::WallUid;
 pub use v3::Attempt;
-pub use v3::UserInteraction;
 pub use v4::DatedAttempt;
+pub use v4::Pattern;
+pub use v4::Problem;
+pub use v4::Transformation;
+pub use v4::UserInteraction;
+pub use v4::Wall;
 
 mod semantics;
 
 pub mod v4 {
+    use super::v1;
+    use super::v2;
     use super::v3;
     use chrono::DateTime;
     use chrono::Utc;
+    use serde::Deserialize;
+    use serde::Serialize;
+    use std::collections::BTreeMap;
+    use std::collections::BTreeSet;
+
+    #[derive(Serialize, Deserialize, Debug, Clone)]
+    pub struct Wall {
+        pub uid: v2::WallUid,
+        pub wall_dimensions: v2::WallDimensions,
+        pub holds: BTreeMap<v1::HoldPosition, v2::Hold>,
+
+        /// Canonicalized.
+        pub problems: BTreeSet<Problem>,
+    }
+
+    #[derive(Serialize, Deserialize, Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
+    pub struct Problem {
+        pub pattern: Pattern,
+        pub method: v2::Method,
+    }
+
+    #[derive(Serialize, Deserialize, Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Hash)]
+    pub struct Pattern {
+        pub pattern: BTreeMap<v1::HoldPosition, v1::HoldRole>,
+    }
+
+    #[derive(Serialize, Deserialize, Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Copy, Default)]
+    pub struct Transformation {
+        pub shift_right: u64,
+        pub mirror: bool,
+    }
+
+    #[derive(Serialize, Deserialize, Debug, Clone)]
+    pub struct UserInteraction {
+        pub wall_uid: v2::WallUid,
+        pub problem: Problem,
+
+        /// Dates on which this problem was attempted, and how it went
+        pub attempted_on: BTreeMap<chrono::DateTime<chrono::Utc>, v3::Attempt>,
+
+        /// Is among favorite problems
+        pub is_favorite: bool,
+    }
 
     #[derive(Debug, Clone, Copy)]
     pub struct DatedAttempt {
@@ -85,7 +131,6 @@ pub mod v2 {
     pub struct Wall {
         pub uid: WallUid,
 
-        // TODO: Replace by walldimensions
         pub rows: u64,
         pub cols: u64,
 
@@ -93,7 +138,7 @@ pub mod v2 {
         pub problems: BTreeSet<ProblemUid>,
     }
 
-    #[derive(Debug, Copy, Clone, PartialEq, Eq)]
+    #[derive(Debug, Copy, Clone, PartialEq, Eq, Serialize, Deserialize)]
     pub struct WallDimensions {
         pub rows: u64,
         pub cols: u64,
@@ -115,16 +160,16 @@ pub mod v2 {
     #[derive(Serialize, Deserialize, Debug, PartialEq, Eq, PartialOrd, Ord, Clone, Copy, Hash, derive_more::FromStr, derive_more::Display)]
     pub struct ProblemUid(pub uuid::Uuid);
 
-    #[derive(Serialize, Deserialize, Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Display)]
+    #[derive(Serialize, Deserialize, Debug, Clone, PartialEq, Eq, PartialOrd, Ord, Display, Copy, Hash)]
     pub enum Method {
         #[display("Feet follow hands")]
         FeetFollowHands,
 
-        #[display("Footless")]
-        Footless,
-
         #[display("Footless plus kickboard")]
         FootlessPlusKickboard,
+
+        #[display("Footless")]
+        Footless,
     }
 
     #[derive(Serialize, Deserialize, Clone, Debug)]

crates/ascend/src/models/semantics.rs

@@ -1,16 +1,136 @@
 use super::*;
+use crate::gradient::Gradient;
 use chrono::DateTime;
 use chrono::Utc;
 use std::collections::BTreeMap;
+use std::collections::HashSet;
+
+impl Problem {
+    /// Returns all possible transformations for the pattern. Not for the method.
+    #[must_use]
+    pub fn transformations(&self, wall_dimensions: WallDimensions) -> HashSet<Self> {
+        self.pattern
+            .transformations(wall_dimensions)
+            .into_iter()
+            .map(|pattern| Self {
+                pattern,
+                method: self.method,
+            })
+            .collect()
+    }
+}
+
+impl Pattern {
+    #[must_use]
+    pub fn canonicalize(&self) -> Self {
+        let mut pattern = self.clone();
+        let min_col = pattern.pattern.keys().map(|hold_position| hold_position.col).min().unwrap_or(0);
+        pattern.pattern = pattern
+            .pattern
+            .iter()
+            .map(|(hold_position, hold_role)| {
+                let hold_position = HoldPosition {
+                    row: hold_position.row,
+                    col: hold_position.col - min_col,
+                };
+                (hold_position, *hold_role)
+            })
+            .collect();
+        std::cmp::min(pattern.mirror(), pattern)
+    }
+
+    #[must_use]
+    pub fn shift_left(&self, shift: u64) -> Option<Self> {
+        // Out of bounds check
+        if let Some(min_col) = self.pattern.keys().map(|hold_position| hold_position.col).min() {
+            if shift > min_col {
+                return None;
+            }
+        }
+
+        let pattern: BTreeMap<HoldPosition, HoldRole> = self
+            .pattern
+            .iter()
+            .map(|(hold_position, hold_role)| {
+                let mut hold_position = *hold_position;
+                hold_position.col -= shift;
+                (hold_position, *hold_role)
+            })
+            .collect();
+
+        Some(Self { pattern })
+    }
+
+    #[must_use]
+    pub fn shift_right(&self, wall_dimensions: WallDimensions, shift: u64) -> Option<Self> {
+        // Out of bounds check
+        if let Some(max_col) = self.pattern.keys().map(|hold_position| hold_position.col).max() {
+            if max_col + shift >= wall_dimensions.cols {
+                return None;
+            }
+        }
+
+        let pattern: BTreeMap<HoldPosition, HoldRole> = self
+            .pattern
+            .iter()
+            .map(|(hold_position, hold_role)| {
+                let mut hold_position = *hold_position;
+                hold_position.col += shift;
+                (hold_position, *hold_role)
+            })
+            .collect();
+
+        Some(Self { pattern })
+    }
+
+    #[must_use]
+    pub fn mirror(&self) -> Self {
+        let mut pattern = self.clone();
+
+        let min_col = pattern.pattern.keys().map(|hold_position| hold_position.col).min().unwrap_or(0);
+        let max_col = pattern.pattern.keys().map(|hold_position| hold_position.col).max().unwrap_or(0);
+
+        pattern.pattern = pattern
+            .pattern
+            .iter()
+            .map(|(hold_position, hold_role)| {
+                let HoldPosition { row, col } = *hold_position;
+                let mut mirrored_col = col;
+                mirrored_col += 2 * (max_col - col);
+                mirrored_col -= max_col - min_col;
+                let hold_position = HoldPosition { row, col: mirrored_col };
+                (hold_position, *hold_role)
+            })
+            .collect();
+
+        pattern
+    }
+
+    /// Returns all possible transformations for the pattern
+    #[must_use]
+    pub fn transformations(&self, wall_dimensions: WallDimensions) -> HashSet<Self> {
+        let mut transformations = HashSet::new();
+
+        let pattern = self.canonicalize();
+        for mut pat in [pattern.mirror(), pattern] {
+            transformations.insert(pat.clone());
+            while let Some(p) = pat.shift_right(wall_dimensions, 1) {
+                transformations.insert(p.clone());
+                pat = p;
+            }
+        }
+
+        transformations
+    }
+}
 
 impl UserInteraction {
-    pub(crate) fn new(wall_uid: WallUid, problem_uid: ProblemUid) -> Self {
+    pub(crate) fn new(wall_uid: WallUid, problem: Problem) -> Self {
         Self {
             wall_uid,
-            problem_uid,
+            problem,
             is_favorite: false,
             attempted_on: BTreeMap::new(),
-            is_saved: false,
         }
     }
 
@@ -63,15 +183,11 @@ impl WallUid {
     pub(crate) fn create() -> Self {
         Self(uuid::Uuid::new_v4())
     }
-}
-
-impl ProblemUid {
-    pub(crate) fn create() -> Self {
-        Self(uuid::Uuid::new_v4())
-    }
+    #[expect(dead_code)]
     pub(crate) fn min() -> Self {
         Self(uuid::Uuid::nil())
     }
+    #[expect(dead_code)]
     pub(crate) fn max() -> Self {
         Self(uuid::Uuid::max())
     }
@@ -102,4 +218,78 @@ impl Attempt {
             Attempt::Flash => Icon::BoltSolid,
         }
     }
+
+    pub(crate) fn gradient(&self) -> Gradient {
+        match self {
+            Attempt::Attempt => Gradient::PinkOrange,
+            Attempt::Send => Gradient::TealLime,
+            Attempt::Flash => Gradient::CyanBlue,
+        }
+    }
+}
+
+impl std::str::FromStr for Problem {
+    type Err = ron::Error;
+    fn from_str(s: &str) -> Result<Self, Self::Err> {
+        let problem = ron::from_str(s)?;
+        Ok(problem)
+    }
+}
+
+impl std::fmt::Display for Problem {
+    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
+        let s = ron::to_string(self).unwrap();
+        write!(f, "{s}")?;
+        Ok(())
+    }
+}
+
+#[cfg(test)]
+mod tests {
+    use super::*;
+
+    #[test_try::test_try]
+    fn canonicalize_empty_pattern() {
+        let pattern = Pattern {
+            pattern: [].into_iter().collect(),
+        };
+        let canonicalized = pattern.canonicalize();
+        assert_eq!(pattern, canonicalized);
+        let mirrored = pattern.mirror();
+        assert_eq!(pattern, mirrored);
+    }
+
+    #[test_try::test_try]
+    fn canonicalize_pattern() {
+        let pattern = Pattern {
+            pattern: [
+                (HoldPosition { row: 0, col: 1 }, HoldRole::End),
+                (HoldPosition { row: 7, col: 6 }, HoldRole::Start),
+            ]
+            .into_iter()
+            .collect(),
+        };
+
+        let canonicalized = pattern.canonicalize();
+
+        assert_eq!(canonicalized.pattern[&HoldPosition { row: 0, col: 0 }], HoldRole::End);
+        assert_eq!(canonicalized.pattern[&HoldPosition { row: 7, col: 5 }], HoldRole::Start);
+    }
+
+    #[test_try::test_try]
+    fn mirror_pattern() {
+        let pattern = Pattern {
+            pattern: [
+                (HoldPosition { row: 0, col: 1 }, HoldRole::End),
+                (HoldPosition { row: 7, col: 6 }, HoldRole::Start),
+            ]
+            .into_iter()
+            .collect(),
+        };
+
+        let mirrored = pattern.mirror();
+
+        assert_eq!(mirrored.pattern[&HoldPosition { row: 0, col: 6 }], HoldRole::End);
+        assert_eq!(mirrored.pattern[&HoldPosition { row: 7, col: 1 }], HoldRole::Start);
+    }
 }

crates/ascend/src/pages.rs

@@ -1,4 +1,3 @@
-pub mod edit_wall;
-pub mod routes;
+pub mod holds;
 pub mod settings;
 pub mod wall;

crates/ascend/src/pages/holds.rs

@@ -24,7 +24,7 @@ struct RouteParams {
 }
 
 #[component]
-pub fn EditWall() -> impl IntoView {
+pub fn Page() -> impl IntoView {
     let params = leptos_router::hooks::use_params::<RouteParams>();
     let wall_uid = Signal::derive(move || {
         params
@@ -85,8 +85,8 @@ fn Ready(wall: models::Wall) -> impl IntoView {
     }
 
     let style = {
-        let grid_rows = crate::css::grid_rows_n(wall.rows);
-        let grid_cols = crate::css::grid_cols_n(wall.cols);
+        let grid_rows = crate::css::grid_rows_n(wall.wall_dimensions.rows);
+        let grid_cols = crate::css::grid_cols_n(wall.wall_dimensions.cols);
         [grid_rows, grid_cols].join(" ")
     };
 

crates/ascend/src/pages/routes.rs

@@ -1,102 +0,0 @@
-use crate::components;
-use crate::components::header::HeaderItem;
-use crate::components::header::HeaderItems;
-use crate::components::header::StyledHeader;
-use crate::models;
-use leptos::Params;
-use leptos::prelude::*;
-use leptos_router::params::Params;
-
-#[derive(Params, PartialEq, Clone)]
-struct RouteParams {
-    // Is never None
-    wall_uid: Option<models::WallUid>,
-}
-
-#[component]
-#[tracing::instrument(skip_all)]
-pub fn Routes() -> impl IntoView {
-    tracing::debug!("Enter");
-
-    let params = leptos_router::hooks::use_params::<RouteParams>();
-    let wall_uid = Signal::derive(move || {
-        params
-            .get()
-            .expect("gets wall_uid from URL")
-            .wall_uid
-            .expect("wall_uid param is never None")
-    });
-
-    let wall = crate::resources::wall_by_uid(wall_uid);
-    let problems = crate::resources::problems_for_wall(wall_uid);
-
-    let header_items = move || HeaderItems {
-        left: vec![HeaderItem {
-            text: "← Ascend".to_string(),
-            link: Some(format!("/wall/{}", wall_uid.get())),
-        }],
-        middle: vec![HeaderItem {
-            text: "Routes".to_string(),
-            link: None,
-        }],
-        right: vec![],
-    };
-
-    let suspend = move || {
-        Suspend::new(async move {
-            let wall = wall.await;
-            let problems = problems.await;
-
-            let v = move || -> Result<_, ServerFnError> {
-                let wall = wall.clone()?;
-                let problems = problems.clone()?;
-
-                let wall_dimensions = models::WallDimensions {
-                    rows: wall.rows,
-                    cols: wall.cols,
-                };
-                let problems_sample = move || problems.values().take(10).cloned().collect::<Vec<_>>();
-
-                Ok(view! {
-                    <div>
-                        <For
-                            each=problems_sample
-                            key=|problem| problem.uid
-                            children=move |problem: models::Problem| {
-                                view! {
-                                    <Problem dim=wall_dimensions problem />
-                                    <hr class="my-8 h-px bg-gray-700 border-0" />
-                                }
-                            }
-                        />
-                    </div>
-                })
-            };
-
-            view! { <ErrorBoundary fallback=|_errors| "error">{v}</ErrorBoundary> }
-        })
-    };
-
-    view! {
-        <div class="min-h-screen min-w-screen bg-neutral-950">
-            <StyledHeader items=Signal::derive(header_items) />
-
-            <div class="container mx-auto mt-6">
-                <Suspense fallback=|| view! { <p>"loading"</p> }>{suspend}</Suspense>
-            </div>
-        </div>
-    }
-}
-
-#[component]
-#[tracing::instrument(skip_all)]
-fn Problem(#[prop(into)] dim: Signal<models::WallDimensions>, #[prop(into)] problem: Signal<models::Problem>) -> impl IntoView {
-    view! {
-        <div class="flex items-start">
-            <div class="flex-none">
-                <components::Problem dim problem />
-            </div>
-            <components::ProblemInfo problem=problem.get() />
-        </div>
-    }
-}

crates/ascend/src/pages/settings.rs

@@ -29,20 +29,18 @@ pub fn Settings() -> impl IntoView {
     }
 }
 
-#[component]
-#[tracing::instrument(skip_all)]
-fn Import(wall_uid: WallUid) -> impl IntoView {
-    let import_from_mini_moonboard = ServerAction::<ImportFromMiniMoonboard>::new();
-
-    let onclick = move |_mouse_event| {
-        import_from_mini_moonboard.dispatch(ImportFromMiniMoonboard { wall_uid });
-    };
-
-    view! {
-        <p>"Import problems from"</p>
-        <button on:click=onclick>"Mini Moonboard"</button>
-    }
-}
+// #[component]
+// #[tracing::instrument(skip_all)]
+// fn Import(wall_uid: WallUid) -> impl IntoView {
+//     let import_from_mini_moonboard = ServerAction::<ImportFromMiniMoonboard>::new();
+//     let onclick = move |_mouse_event| {
+//         import_from_mini_moonboard.dispatch(ImportFromMiniMoonboard { wall_uid });
+//     };
+//     view! {
+//         <p>"Import problems from"</p>
+//         <button on:click=onclick>"Mini Moonboard"</button>
+//     }
+// }
 
 #[server]
 #[tracing::instrument]

crates/ascend/src/pages/wall.rs

@@ -8,11 +8,14 @@ use crate::gradient::Gradient;
 use crate::models;
 use crate::models::HoldRole;
 use crate::server_functions;
+use crate::server_functions::SetIsFavorite;
 use leptos::Params;
 use leptos::prelude::*;
 use leptos_router::params::Params;
 use std::collections::BTreeMap;
 use std::collections::BTreeSet;
+use std::collections::HashSet;
+use std::ops::Deref;
 
 #[derive(Params, PartialEq, Clone)]
 struct RouteParams {
@@ -35,8 +38,7 @@ pub fn Page() -> impl IntoView {
     });
 
     let wall = crate::resources::wall_by_uid(wall_uid);
-    let problems = crate::resources::problems_for_wall(wall_uid);
-    let user_interactions = crate::resources::user_interactions(wall_uid);
+    let user_interactions = crate::resources::user_interactions_for_wall(wall_uid);
 
     leptos::view! {
         <div class="min-h-screen min-w-screen bg-neutral-950">
@@ -46,10 +48,9 @@ pub fn Page() -> impl IntoView {
                 {move || Suspend::new(async move {
                     tracing::debug!("executing main suspend");
                     let wall = wall.await?;
-                    let problems = problems.await?;
                     let user_interactions = user_interactions.await?;
                     let user_interactions = RwSignal::new(user_interactions);
-                    Ok::<_, ServerFnError>(view! { <Controller wall problems user_interactions /> })
+                    Ok::<_, ServerFnError>(view! { <Controller wall user_interactions /> })
                 })}
             </Suspense>
         </div>
@@ -59,9 +60,9 @@ pub fn Page() -> impl IntoView {
 #[derive(Debug, Clone, Copy)]
 struct Context {
     wall: Signal<models::Wall>,
-    user_interactions: Signal<BTreeMap<models::ProblemUid, models::UserInteraction>>,
+    user_interactions: Signal<BTreeMap<models::Problem, models::UserInteraction>>,
     problem: Signal<Option<models::Problem>>,
-    filtered_problems: Signal<BTreeMap<models::ProblemUid, models::Problem>>,
+    filtered_problem_transformations: Signal<Vec<HashSet<models::Problem>>>,
     user_interaction: Signal<Option<models::UserInteraction>>,
     todays_attempt: Signal<Option<models::Attempt>>,
     latest_attempt: Signal<Option<models::DatedAttempt>>,
@@ -69,20 +70,21 @@ struct Context {
     cb_click_hold: Callback<models::HoldPosition>,
     cb_remove_hold_from_filter: Callback<models::HoldPosition>,
     cb_next_problem: Callback<()>,
+    cb_set_problem: Callback<models::Problem>,
     cb_upsert_todays_attempt: Callback<server_functions::UpsertTodaysAttempt>,
+    cb_set_is_favorite: Callback<bool>,
 }
 
 #[component]
 #[tracing::instrument(skip_all)]
 fn Controller(
     #[prop(into)] wall: Signal<models::Wall>,
-    #[prop(into)] problems: Signal<BTreeMap<models::ProblemUid, models::Problem>>,
-    #[prop(into)] user_interactions: RwSignal<BTreeMap<models::ProblemUid, models::UserInteraction>>,
+    #[prop(into)] user_interactions: RwSignal<BTreeMap<models::Problem, models::UserInteraction>>,
 ) -> impl IntoView {
     crate::tracing::on_enter!();
 
     // Extract data from URL
-    let (problem_uid, set_problem_uid) = leptos_router::hooks::query_signal::<models::ProblemUid>("problem");
+    let (problem, set_problem) = leptos_router::hooks::query_signal::<models::Problem>("problem");
 
     // Filter
     let (filter_holds, set_filter_holds) = signal(BTreeSet::new());
@@ -93,10 +95,10 @@ fn Controller(
     });
 
     // Derive signals
-    let wall_uid = signals::wall_uid(wall);
-    let problem = signals::problem(problems, problem_uid.into());
-    let user_interaction = signals::user_interaction(user_interactions.into(), problem_uid.into());
-    let filtered_problems = signals::filtered_problems(problems, filter_holds.into());
+    let user_interaction = signals::user_interaction(user_interactions.into(), problem.into());
+    let problem_transformations = signals::problem_transformations(wall);
+
+    let filtered_problem_transformations = signals::filtered_problem_transformations(problem_transformations.into(), filter_holds.into());
     let todays_attempt = signals::todays_attempt(user_interaction);
     let latest_attempt = signals::latest_attempt(user_interaction);
 
@@ -106,17 +108,45 @@ fn Controller(
         upsert_todays_attempt.dispatch(RonEncoded(attempt));
     });
 
+    // Set favorite
+    let set_is_favorite = ServerAction::<RonEncoded<server_functions::SetIsFavorite>>::new();
+    let cb_set_is_favorite = Callback::new(move |is_favorite| {
+        let wall_uid = wall.read().uid;
+        let Some(problem) = problem.get() else {
+            return;
+        };
+        set_is_favorite.dispatch(RonEncoded(SetIsFavorite {
+            wall_uid,
+            problem,
+            is_favorite,
+        }));
+    });
+
+    // Callback: Set specific problem
+    let cb_set_problem: Callback<models::Problem> = Callback::new(move |problem| {
+        set_problem.set(Some(problem));
+    });
+
     // Callback: Set next problem to a random problem
     let cb_set_random_problem: Callback<()> = Callback::new(move |_| {
         // TODO: remove current problem from population
-        let population = filtered_problems.read();
-        let population = population.keys().copied();
+        let population = filtered_problem_transformations.read();
+        let population = population.deref();
 
         use rand::seq::IteratorRandom;
         let mut rng = rand::rng();
-        let problem_uid = population.choose(&mut rng);
 
-        set_problem_uid.set(problem_uid);
+        // Pick pattern
+        let Some(problem_set) = population.iter().choose(&mut rng) else {
+            return;
+        };
+
+        // Pick problem out of pattern transformations
+        let Some(problem) = problem_set.iter().choose(&mut rng) else {
+            return;
+        };
+
+        set_problem.set(Some(problem.clone()));
     });
 
     // Callback: On click hold, Add/Remove hold position to problem filter
@@ -130,7 +160,7 @@ fn Controller(
 
     // Set a problem when wall is set (loaded)
     Effect::new(move |_prev_value| {
-        if problem_uid.get().is_none() {
+        if problem.read().is_none() {
             tracing::debug!("Setting initial problem");
             cb_set_random_problem.run(());
         }
@@ -141,23 +171,35 @@ fn Controller(
         if let Some(Ok(v)) = upsert_todays_attempt.value().get() {
             let v = v.into_inner();
             user_interactions.update(|map| {
-                map.insert(v.problem_uid, v);
+                map.insert(v.problem.clone(), v);
+            });
+        }
+    });
+
+    // Update user interactions after setting favorite
+    Effect::new(move || {
+        if let Some(Ok(v)) = set_is_favorite.value().get() {
+            let v = v.into_inner();
+            user_interactions.update(|map| {
+                map.insert(v.problem.clone(), v);
             });
         }
     });
 
     provide_context(Context {
         wall,
-        problem,
+        problem: problem.into(),
         cb_click_hold,
         user_interaction,
         latest_attempt,
         cb_upsert_todays_attempt,
         cb_remove_hold_from_filter,
         cb_next_problem: cb_set_random_problem,
+        cb_set_problem,
+        cb_set_is_favorite,
         todays_attempt,
         filter_holds: filter_holds.into(),
-        filtered_problems: filtered_problems.into(),
+        filtered_problem_transformations: filtered_problem_transformations.into(),
         user_interactions: user_interactions.into(),
     });
 
@@ -183,8 +225,7 @@ fn View() -> impl IntoView {
 
                     <Separator />
 
-                    <div class="flex flex-row justify-around">
-                        <Transformations />
+                    <div class="flex flex-row justify-between">
                         <NextProblemButton />
                     </div>
                 </Section>
@@ -193,11 +234,14 @@ fn View() -> impl IntoView {
 
                 <Section title="Current problem">
                     {move || ctx.problem.get().map(|problem| view! { <ProblemInfo problem /> })}
-                    <Separator /> <AttemptRadioGroup /> <Separator /> <History />
+                    <Separator /> <div class="flex flex-row gap-2 justify-between">
+                        <Transformations />
+                        <FavoriteButton />
+                    </div> <Separator /> <AttemptRadioGroup /> <Separator /> <History />
                 </Section>
             </div>
 
-            <div class="flex-auto flex flex-row justify-end items-start px-2 pt-3">
+            <div class="flex flex-col px-2 pt-3 gap-4">
                 <HoldsButton />
             </div>
         </div>
@@ -211,25 +255,63 @@ fn Transformations() -> impl IntoView {
 
     let ctx = use_context::<Context>().unwrap();
 
-    let on_left = Callback::new(move |()| {
-        tracing::info!("left");
-    });
-    let on_mirror = Callback::new(move |()| {
-        tracing::info!("mirror");
-    });
-    let on_right = Callback::new(move |()| {
-        tracing::info!("right");
+    let left = Signal::derive(move || {
+        let mut problem = ctx.problem.get()?;
+        let new_pattern = problem.pattern.shift_left(1)?;
+        problem.pattern = new_pattern;
+        Some(problem)
     });
 
+    let right = Signal::derive(move || {
+        let mut problem = ctx.problem.get()?;
+        let wall_dimensions = ctx.wall.read().wall_dimensions;
+        let new_pattern = problem.pattern.shift_right(wall_dimensions, 1)?;
+        problem.pattern = new_pattern;
+        Some(problem)
+    });
+
+    let on_click_left = Callback::new(move |()| {
+        tracing::debug!("left");
+        if let Some(problem) = left.get() {
+            ctx.cb_set_problem.run(problem);
+        }
+    });
+    let on_click_mirror = Callback::new(move |()| {
+        tracing::debug!("mirror");
+        if let Some(mut problem) = ctx.problem.get() {
+            problem.pattern = problem.pattern.mirror();
+            ctx.cb_set_problem.run(problem);
+        }
+    });
+    let on_click_right = Callback::new(move |()| {
+        tracing::debug!("right");
+        if let Some(problem) = right.get() {
+            ctx.cb_set_problem.run(problem);
+        }
+    });
+
+    let left_disabled = Signal::derive(move || left.read().is_none());
+    let right_disabled = Signal::derive(move || right.read().is_none());
+
     view! {
-        <div class="flex flex-row justify-center gap-2">
-            <Button icon=Icon::ChevronLeft disabled=true on_click=on_left />
-            <Button icon=Icon::CodeBracketSquare on_click=on_mirror />
-            <Button icon=Icon::ChevronRight on_click=on_right />
+        <div class="flex flex-row gap-2 justify-center">
+            <Button icon=Icon::ChevronLeft disabled=left_disabled on_click=on_click_left />
+            <Button icon=Icon::CodeBracketSquare on_click=on_click_mirror />
+            <Button icon=Icon::ChevronRight disabled=right_disabled on_click=on_click_right />
         </div>
     }
 }
 
+#[component]
+#[tracing::instrument(skip_all)]
+fn LikedButton() -> impl IntoView {
+    crate::tracing::on_enter!();
+
+    let _ctx = use_context::<Context>().unwrap();
+
+    view! { <Button text="Saved" icon=Icon::HeartOutline color=Gradient::PinkRed /> }
+}
+
 #[component]
 #[tracing::instrument(skip_all)]
 fn HoldsButton() -> impl IntoView {
@@ -246,6 +328,25 @@ fn HoldsButton() -> impl IntoView {
     }
 }
 
+#[component]
+#[tracing::instrument(skip_all)]
+fn FavoriteButton() -> impl IntoView {
+    crate::tracing::on_enter!();
+
+    let ctx = use_context::<Context>().unwrap();
+
+    let ui_toggle = Signal::derive(move || {
+        let guard = ctx.user_interaction.read();
+        guard.as_ref().map(|user_interaction| user_interaction.is_favorite).unwrap_or(false)
+    });
+    let on_click = Callback::new(move |_| {
+        ctx.cb_set_is_favorite.run(!ui_toggle.get());
+    });
+    let icon = Signal::derive(move || if ui_toggle.get() { Icon::Heart } else { Icon::HeartOutline });
+    let text = Signal::derive(move || if ui_toggle.get() { "Saved" } else { "Save" }.to_string());
+    view! { <Button text icon on_click color=Gradient::PinkRed /> }
+}
+
 #[component]
 #[tracing::instrument(skip_all)]
 fn NextProblemButton() -> impl IntoView {
@@ -254,7 +355,7 @@ fn NextProblemButton() -> impl IntoView {
     let ctx = use_context::<Context>().unwrap();
 
     let on_click = Callback::new(move |_| ctx.cb_next_problem.run(()));
-    view! { <Button icon=Icon::ArrowPath text="Next problem" on_click color=Gradient::PurpleBlue /> }
+    view! { <Button icon=Icon::ArrowPath text="Randomize" on_click color=Gradient::PurpleBlue /> }
 }
 
 #[component]
@@ -284,11 +385,13 @@ fn Filter() -> impl IntoView {
             }
         }
 
+        let problems_count = ctx.filtered_problem_transformations.read().iter().map(|set| set.len()).sum::<usize>();
+
         let problems_counter = {
-            let name = view! { <p class="font-light mr-4 text-right text-orange-300">{"Problems:"}</p> };
-            let value = view! { <p class="text-white">{ctx.filtered_problems.read().len()}</p> };
+            let name = view! { <p class="mr-4 font-light text-right text-orange-300">{"Problems:"}</p> };
+            let value = view! { <p class="text-white">{problems_count}</p> };
             view! {
-                <div class="grid grid-rows-none gap-y-1 gap-x-0.5 grid-cols-[auto_1fr]">
+                <div class="grid grid-rows-none gap-x-0.5 gap-y-1 grid-cols-[auto_1fr]">
                     {name} {value}
                 </div>
             }
@@ -305,13 +408,15 @@ fn Filter() -> impl IntoView {
         let mut interaction_counters = InteractionCounters::default();
         let interaction_counters_view = {
             let user_ints = ctx.user_interactions.read();
-            for problem_uid in ctx.filtered_problems.read().keys() {
-                if let Some(user_int) = user_ints.get(problem_uid) {
-                    match user_int.best_attempt().map(|da| da.attempt) {
-                        Some(models::Attempt::Flash) => interaction_counters.flash += 1,
-                        Some(models::Attempt::Send) => interaction_counters.send += 1,
-                        Some(models::Attempt::Attempt) => interaction_counters.attempt += 1,
-                        None => {}
+            for problem_set in ctx.filtered_problem_transformations.read().iter() {
+                for problem in problem_set {
+                    if let Some(user_int) = user_ints.get(problem) {
+                        match user_int.best_attempt().map(|da| da.attempt) {
+                            Some(models::Attempt::Flash) => interaction_counters.flash += 1,
+                            Some(models::Attempt::Send) => interaction_counters.send += 1,
+                            Some(models::Attempt::Attempt) => interaction_counters.attempt += 1,
+                            None => {}
+                        }
                     }
                 }
             }
@@ -370,7 +475,7 @@ fn AttemptRadioGroup() -> impl IntoView {
 
     let ctx = use_context::<Context>().unwrap();
 
-    let problem_uid = Signal::derive(move || ctx.problem.read().as_ref().map(|p| p.uid));
+    let problem = ctx.problem;
 
     let mut attempt_radio_buttons = vec![];
     for variant in [models::Attempt::Flash, models::Attempt::Send, models::Attempt::Attempt] {
@@ -379,10 +484,10 @@ fn AttemptRadioGroup() -> impl IntoView {
         let onclick = move |_| {
             let attempt = if ui_toggle.get() { None } else { Some(variant) };
 
-            if let Some(problem_uid) = problem_uid.get() {
+            if let Some(problem) = problem.get() {
                 ctx.cb_upsert_todays_attempt.run(server_functions::UpsertTodaysAttempt {
                     wall_uid: ctx.wall.read().uid,
-                    problem_uid,
+                    problem,
                     attempt,
                 });
             }
@@ -390,7 +495,7 @@ fn AttemptRadioGroup() -> impl IntoView {
         attempt_radio_buttons.push(view! { <AttemptRadioButton on:click=onclick variant selected=ui_toggle /> });
     }
 
-    view! { <div class="gap-2 flex flex-col justify-evenly 2xl:flex-row">{attempt_radio_buttons}</div> }
+    view! { <div class="flex flex-col gap-2 justify-evenly 2xl:flex-row">{attempt_radio_buttons}</div> }
 }
 
 #[component]
@@ -469,14 +574,14 @@ fn Wall() -> impl IntoView {
         }
 
         let style = {
-            let grid_rows = crate::css::grid_rows_n(wall.rows);
-            let grid_cols = crate::css::grid_cols_n(wall.cols);
-            let max_width = format!("{}vh", wall.cols as f64 / wall.rows as f64 * 100.);
+            let grid_rows = crate::css::grid_rows_n(wall.wall_dimensions.rows);
+            let grid_cols = crate::css::grid_cols_n(wall.wall_dimensions.cols);
+            let max_width = format!("{}vh", wall.wall_dimensions.cols as f64 / wall.wall_dimensions.rows as f64 * 100.);
             format!("max-height: 100vh; max-width: {max_width}; {}", [grid_rows, grid_cols].join(" "))
         };
 
         view! {
-            <div style=style class="p-1 grid gap-1">
+            <div style=style class="grid gap-1 p-1">
                 {cells}
             </div>
         }
@@ -524,13 +629,13 @@ fn Hold(
 
 #[component]
 fn Separator() -> impl IntoView {
-    view! { <div class="m-2 sm:m-3 md:m-4 h-4" /> }
+    view! { <div class="m-2 h-4 sm:m-3 md:m-4" /> }
 }
 
 #[component]
 fn Section(children: Children, #[prop(into)] title: MaybeProp<String>) -> impl IntoView {
     view! {
-        <div class="bg-neutral-900 px-5 pt-3 pb-8 rounded-lg">
+        <div class="px-5 pt-3 pb-8 rounded-lg bg-neutral-900">
             <div class="py-3 text-lg text-center text-orange-400 border-t border-orange-400">
                 {move || title.get()}
             </div>
@@ -544,6 +649,7 @@ mod signals {
     use leptos::prelude::*;
     use std::collections::BTreeMap;
     use std::collections::BTreeSet;
+    use std::collections::HashSet;
 
     pub fn latest_attempt(user_interaction: Signal<Option<models::UserInteraction>>) -> Signal<Option<models::DatedAttempt>> {
         Signal::derive(move || user_interaction.read().as_ref().and_then(models::UserInteraction::latest_attempt))
@@ -553,49 +659,73 @@ mod signals {
         Signal::derive(move || latest_attempt.read().as_ref().and_then(models::UserInteraction::todays_attempt))
     }
 
+    #[expect(dead_code)]
     pub fn wall_uid(wall: Signal<models::Wall>) -> Signal<models::WallUid> {
         Signal::derive(move || wall.read().uid)
     }
 
     pub fn user_interaction(
-        user_interactions: Signal<BTreeMap<models::ProblemUid, models::UserInteraction>>,
-        problem_uid: Signal<Option<models::ProblemUid>>,
+        user_interactions: Signal<BTreeMap<models::Problem, models::UserInteraction>>,
+        problem: Signal<Option<models::Problem>>,
     ) -> Signal<Option<models::UserInteraction>> {
         Signal::derive(move || {
-            let problem_uid = problem_uid.get()?;
+            let problem = problem.get()?;
             let user_interactions = user_interactions.read();
-            user_interactions.get(&problem_uid).cloned()
+            user_interactions.get(&problem).cloned()
         })
     }
 
-    pub(crate) fn problem(
-        problems: Signal<BTreeMap<models::ProblemUid, models::Problem>>,
-        problem_uid: Signal<Option<models::ProblemUid>>,
-    ) -> Signal<Option<models::Problem>> {
-        Signal::derive(move || {
-            let problem_uid = problem_uid.get()?;
-            let problems = problems.read();
-            problems.get(&problem_uid).cloned()
+    /// Maps each problem to a set of problems comprising all transformation of the problem pattern.
+    pub(crate) fn problem_transformations(wall: Signal<models::Wall>) -> Memo<Vec<HashSet<models::Problem>>> {
+        Memo::new(move |_prev_val| {
+            let wall = wall.read();
+            wall.problems
+                .iter()
+                .map(|problem| problem.transformations(wall.wall_dimensions))
+                .collect()
         })
     }
 
+    #[expect(dead_code)]
     pub(crate) fn filtered_problems(
-        problems: Signal<BTreeMap<models::ProblemUid, models::Problem>>,
+        wall: Signal<models::Wall>,
         filter_holds: Signal<BTreeSet<models::HoldPosition>>,
-    ) -> Memo<BTreeMap<models::ProblemUid, models::Problem>> {
+    ) -> Memo<BTreeSet<models::Problem>> {
         Memo::new(move |_prev_val| {
             let filter_holds = filter_holds.read();
-            problems.with(|problems| {
-                problems
+            wall.with(|wall| {
+                wall.problems
                     .iter()
-                    .filter(|(_, problem)| filter_holds.iter().all(|hold_pos| problem.holds.contains_key(hold_pos)))
-                    .map(|(problem_uid, problem)| (*problem_uid, problem.clone()))
-                    .collect::<BTreeMap<models::ProblemUid, models::Problem>>()
+                    .filter(|problem| filter_holds.iter().all(|hold_pos| problem.pattern.pattern.contains_key(hold_pos)))
+                    .cloned()
+                    .collect::<BTreeSet<models::Problem>>()
             })
         })
     }
 
+    pub(crate) fn filtered_problem_transformations(
+        problem_transformations: Signal<Vec<HashSet<models::Problem>>>,
+        filter_holds: Signal<BTreeSet<models::HoldPosition>>,
+    ) -> Memo<Vec<HashSet<models::Problem>>> {
+        Memo::new(move |_prev_val| {
+            let filter_holds = filter_holds.read();
+            let problem_transformations = problem_transformations.read();
+
+            problem_transformations
+                .iter()
+                .map(|problem_set| {
+                    problem_set
+                        .iter()
+                        .filter(|problem| filter_holds.iter().all(|hold_pos| problem.pattern.pattern.contains_key(hold_pos)))
+                        .cloned()
+                        .collect::<HashSet<models::Problem>>()
+                })
+                .filter(|set| !set.is_empty())
+                .collect()
+        })
+    }
+
     pub(crate) fn hold_role(problem: Signal<Option<models::Problem>>, hold_position: models::HoldPosition) -> Signal<Option<models::HoldRole>> {
-        Signal::derive(move || problem.get().and_then(|p| p.holds.get(&hold_position).copied()))
+        Signal::derive(move || problem.get().and_then(|p| p.pattern.pattern.get(&hold_position).copied()))
     }
 }

crates/ascend/src/resources.rs

@@ -17,47 +17,15 @@ pub fn wall_by_uid(wall_uid: Signal<models::WallUid>) -> RonResource<models::Wal
     )
 }
 
-/// Version of [problem_by_uid] that short circuits if the input problem_uid signal is None.
-pub fn problem_by_uid_optional(
-    wall_uid: Signal<models::WallUid>,
-    problem_uid: Signal<Option<models::ProblemUid>>,
-) -> RonResource<Option<models::Problem>> {
-    Resource::new_with_options(
-        move || (wall_uid.get(), problem_uid.get()),
-        move |(wall_uid, problem_uid)| async move {
-            let Some(problem_uid) = problem_uid else {
-                return Ok(None);
-            };
-            crate::server_functions::get_problem_by_uid(wall_uid, problem_uid)
-                .await
-                .map(RonEncoded::into_inner)
-                .map(Some)
-        },
-        false,
-    )
-}
-
-/// Returns all problems for a wall
-pub fn problems_for_wall(wall_uid: Signal<models::WallUid>) -> RonResource<BTreeMap<models::ProblemUid, models::Problem>> {
-    Resource::new_with_options(
-        move || wall_uid.get(),
-        move |wall_uid| async move { crate::server_functions::get_problems_for_wall(wall_uid).await.map(RonEncoded::into_inner) },
-        false,
-    )
-}
-
 /// Returns user interaction for a single problem
-pub fn user_interaction(
-    wall_uid: Signal<models::WallUid>,
-    problem_uid: Signal<Option<models::ProblemUid>>,
-) -> RonResource<Option<models::UserInteraction>> {
+pub fn user_interaction(wall_uid: Signal<models::WallUid>, problem: Signal<Option<models::Problem>>) -> RonResource<Option<models::UserInteraction>> {
     Resource::new_with_options(
-        move || (wall_uid.get(), problem_uid.get()),
-        move |(wall_uid, problem_uid)| async move {
-            let Some(problem_uid) = problem_uid else {
+        move || (wall_uid.get(), problem.get()),
+        move |(wall_uid, problem)| async move {
+            let Some(problem) = problem else {
                 return Ok(None);
             };
-            crate::server_functions::get_user_interaction(wall_uid, problem_uid)
+            crate::server_functions::get_user_interaction(wall_uid, problem)
                 .await
                 .map(RonEncoded::into_inner)
         },
@@ -66,10 +34,14 @@ pub fn user_interaction(
 }
 
 /// Returns all user interactions for a wall
-pub fn user_interactions(wall_uid: Signal<models::WallUid>) -> RonResource<BTreeMap<models::ProblemUid, models::UserInteraction>> {
+pub fn user_interactions_for_wall(wall_uid: Signal<models::WallUid>) -> RonResource<BTreeMap<models::Problem, models::UserInteraction>> {
     Resource::new_with_options(
         move || wall_uid.get(),
-        move |wall_uid| async move { crate::server_functions::get_user_interactions(wall_uid).await.map(RonEncoded::into_inner) },
+        move |wall_uid| async move {
+            crate::server_functions::get_user_interactions_for_wall(wall_uid)
+                .await
+                .map(RonEncoded::into_inner)
+        },
         false,
     )
 }

crates/ascend/src/server/db.rs

@@ -121,13 +121,6 @@ pub async fn init_at_current_version(db: &Database) -> Result<(), DatabaseOperat
                 assert!(table.is_empty()?);
             }
 
-            // Problems table
-            {
-                // Opening the table creates the table
-                let table = txn.open_table(current::TABLE_PROBLEMS)?;
-                assert!(table.is_empty()?);
-            }
-
             // User table
             {
                 // Opening the table creates the table
@@ -147,11 +140,23 @@ use crate::models;
 pub mod current {
     use super::v2;
     use super::v3;
-    pub use v2::TABLE_PROBLEMS;
+    use super::v4;
     pub use v2::TABLE_ROOT;
-    pub use v2::TABLE_WALLS;
-    pub use v3::TABLE_USER;
     pub use v3::VERSION;
+    pub use v4::TABLE_USER;
+    pub use v4::TABLE_WALLS;
+}
+
+pub mod v4 {
+    use crate::models;
+    use crate::server::db::bincode::Bincode;
+    use redb::TableDefinition;
+
+    pub const VERSION: u64 = 4;
+
+    pub const TABLE_WALLS: TableDefinition<Bincode<models::v2::WallUid>, Bincode<models::v4::Wall>> = TableDefinition::new("walls");
+    pub const TABLE_USER: TableDefinition<Bincode<(models::v2::WallUid, models::v4::Problem)>, Bincode<models::v4::UserInteraction>> =
+        TableDefinition::new("user");
 }
 
 pub mod v3 {

crates/ascend/src/server/migrations.rs

@@ -3,12 +3,115 @@ use super::db::DatabaseOperationError;
 use super::db::{self};
 use crate::models;
 use redb::ReadableTable;
+use std::collections::BTreeMap;
 
 #[tracing::instrument(skip_all, err)]
 pub async fn run_migrations(db: &Database) -> Result<(), Box<dyn std::error::Error>> {
     if is_at_version(db, 2).await? {
         migrate_to_v3(db).await?;
     }
+    if is_at_version(db, 3).await? {
+        migrate_to_v4(db).await?;
+    }
+    Ok(())
+}
+
+/// migrate: walls table
+/// migrate: user table
+/// remove: problems table
+#[tracing::instrument(skip_all, err)]
+pub async fn migrate_to_v4(db: &Database) -> Result<(), Box<dyn std::error::Error>> {
+    tracing::warn!("MIGRATING TO VERSION 4");
+
+    db.write(|txn| {
+        let walls_dump = txn
+            .open_table(db::v2::TABLE_WALLS)?
+            .iter()?
+            .map(|el| {
+                let (k, v) = el.unwrap();
+                (k.value(), v.value())
+            })
+            .collect::<BTreeMap<_, _>>();
+        let problems_dump = txn
+            .open_table(db::v2::TABLE_PROBLEMS)?
+            .iter()?
+            .map(|el| {
+                let (k, v) = el.unwrap();
+                (k.value(), v.value())
+            })
+            .collect::<BTreeMap<_, _>>();
+        let user_dump = txn
+            .open_table(db::v3::TABLE_USER)?
+            .iter()?
+            .map(|el| {
+                let (k, v) = el.unwrap();
+                (k.value(), v.value())
+            })
+            .collect::<BTreeMap<_, _>>();
+
+        txn.delete_table(db::v2::TABLE_WALLS)?;
+        txn.delete_table(db::v2::TABLE_PROBLEMS)?;
+        txn.delete_table(db::v3::TABLE_USER)?;
+
+        let mut new_walls_table = txn.open_table(db::current::TABLE_WALLS)?;
+        let mut new_user_table = txn.open_table(db::current::TABLE_USER)?;
+
+        for (wall_uid, wall) in walls_dump.into_iter() {
+            let models::v2::Wall {
+                uid: _,
+                rows,
+                cols,
+                holds,
+                problems,
+            } = wall;
+
+            let problems = problems
+                .into_iter()
+                .map(|problem_uid| {
+                    let old_prob = &problems_dump[&(wall_uid, problem_uid)];
+                    let method = old_prob.method;
+
+                    models::Problem {
+                        pattern: models::Pattern {
+                            pattern: old_prob.holds.clone(),
+                        },
+                        method,
+                    }
+                })
+                .collect();
+            let wall = models::Wall {
+                uid: wall_uid,
+                wall_dimensions: models::WallDimensions { rows, cols },
+                holds,
+                problems,
+            };
+
+            new_walls_table.insert(wall_uid, wall)?;
+        }
+
+        for ((wall_uid, problem_uid), user_interaction) in user_dump.into_iter() {
+            let old_prob = &problems_dump[&(wall_uid, problem_uid)];
+            let problem = models::Problem {
+                pattern: models::Pattern {
+                    pattern: old_prob.holds.clone(),
+                },
+                method: old_prob.method,
+            };
+            let key = (wall_uid, problem.clone());
+            let value = models::UserInteraction {
+                wall_uid,
+                problem,
+                attempted_on: user_interaction.attempted_on,
+                is_favorite: user_interaction.is_favorite,
+            };
+            new_user_table.insert(key, value)?;
+        }
+
+        Ok(())
+    })
+    .await?;
+
+    db.set_version(db::Version { version: db::v4::VERSION }).await?;
 
     Ok(())
 }

crates/ascend/src/server/operations.rs

@@ -20,11 +20,9 @@ pub(crate) async fn import_mini_moonboard_problems(config: &Config, db: Database
 
     tracing::info!("Parsing mini moonboard problems from {file_path}");
 
-    let set_by = "mini-mb-2020-parser";
-
     let mini_moonboard = mini_moonboard::parse(file_path.as_std_path()).await?;
     for mini_mb_problem in mini_moonboard.problems {
-        let mut holds = BTreeMap::<HoldPosition, HoldRole>::new();
+        let mut pattern = BTreeMap::<HoldPosition, HoldRole>::new();
         for mv in mini_mb_problem.moves {
             let row = mv.description.row();
             let col = mv.description.column();
@@ -36,43 +34,27 @@ pub(crate) async fn import_mini_moonboard_problems(config: &Config, db: Database
                 (false, true) => HoldRole::End,
                 (false, false) => HoldRole::Normal,
             };
-            holds.insert(hold_position, role);
+            pattern.insert(hold_position, role);
         }
 
-        let name = mini_mb_problem.name;
-
         let method = match mini_mb_problem.method {
             mini_moonboard::Method::FeetFollowHands => models::Method::FeetFollowHands,
             mini_moonboard::Method::Footless => models::Method::Footless,
             mini_moonboard::Method::FootlessPlusKickboard => models::Method::FootlessPlusKickboard,
         };
 
-        let problem_id = models::ProblemUid::create();
-
-        let problem = models::Problem {
-            uid: problem_id,
-            name,
-            set_by: set_by.to_owned(),
-            holds,
-            method,
-            date_added: chrono::Utc::now(),
-        };
+        let pattern = models::Pattern { pattern }.canonicalize();
+        let problem = models::Problem { pattern, method };
         problems.push(problem);
     }
 
     db.write(|txn| {
         let mut walls_table = txn.open_table(db::current::TABLE_WALLS)?;
-        let mut problems_table = txn.open_table(db::current::TABLE_PROBLEMS)?;
 
         let mut wall = walls_table.get(wall_uid)?.unwrap().value();
-        wall.problems.extend(problems.iter().map(|p| p.uid));
+        wall.problems.extend(problems);
         walls_table.insert(wall_uid, wall)?;
 
-        for problem in problems {
-            let key = (wall_uid, problem.uid);
-            problems_table.insert(key, problem)?;
-        }
-
         Ok(())
     })
     .await?;

crates/ascend/src/server_functions.rs

@@ -9,7 +9,6 @@ use leptos::prelude::*;
 use leptos::server;
 use server_fn::ServerFnError;
 use std::collections::BTreeMap;
-use type_toppings::IteratorExt;
 
 #[server(
     input = Ron,
@@ -21,7 +20,7 @@ pub async fn get_walls() -> Result<RonEncoded<Vec<models::Wall>>, ServerFnError>
     use crate::server::db::Database;
     use leptos::prelude::expect_context;
     use redb::ReadableTable;
-    tracing::trace!("Enter");
+    crate::tracing::on_enter!();
 
     let db = expect_context::<Database>();
 
@@ -46,7 +45,7 @@ pub(crate) async fn get_wall_by_uid(wall_uid: models::WallUid) -> Result<RonEnco
     use crate::server::db::Database;
     use crate::server::db::DatabaseOperationError;
     use leptos::prelude::expect_context;
-    tracing::trace!("Enter");
+    crate::tracing::on_enter!();
 
     #[derive(Debug, derive_more::Error, derive_more::Display)]
     enum Error {
@@ -71,68 +70,6 @@ pub(crate) async fn get_wall_by_uid(wall_uid: models::WallUid) -> Result<RonEnco
     Ok(RonEncoded::new(wall))
 }
 
-#[server(
-    input = Ron,
-    output = Ron,
-    custom = RonEncoded
-)]
-#[tracing::instrument(err(Debug))]
-pub(crate) async fn get_problems_for_wall(
-    wall_uid: models::WallUid,
-) -> Result<RonEncoded<BTreeMap<models::ProblemUid, models::Problem>>, ServerFnError> {
-    use crate::server::db::Database;
-    use crate::server::db::DatabaseOperationError;
-    use leptos::prelude::expect_context;
-    tracing::trace!("Enter");
-
-    #[derive(Debug, derive_more::Error, derive_more::Display, derive_more::From)]
-    enum Error {
-        #[display("Wall not found: {_0:?}")]
-        WallNotFound(#[error(not(source))] models::WallUid),
-
-        DatabaseOperation(DatabaseOperationError),
-    }
-
-    async fn inner(wall_uid: models::WallUid) -> Result<BTreeMap<models::ProblemUid, models::Problem>, Error> {
-        let db = expect_context::<Database>();
-
-        let problems = db
-            .read(|txn| {
-                let walls_table = txn.open_table(crate::server::db::current::TABLE_WALLS)?;
-                tracing::debug!("getting wall");
-                let wall = walls_table
-                    .get(wall_uid)?
-                    .ok_or(Error::WallNotFound(wall_uid))
-                    .map_err(DatabaseOperationError::custom)?
-                    .value();
-                tracing::debug!("got wall");
-                drop(walls_table);
-
-                tracing::debug!("open problems table");
-                let problems_table = txn.open_table(crate::server::db::current::TABLE_PROBLEMS)?;
-                tracing::debug!("opened problems table");
-
-                let problems = wall
-                    .problems
-                    .iter()
-                    .map(|problem_uid| problems_table.get(&(wall_uid, *problem_uid)))
-                    .filter_map(|res| res.transpose())
-                    .map_res(|guard| guard.value())
-                    .map_res(|problem| (problem.uid, problem))
-                    .collect::<Result<BTreeMap<models::ProblemUid, models::Problem>, _>>()?;
-
-                Ok(problems)
-            })
-            .await?;
-
-        Ok(problems)
-    }
-
-    let problems = inner(wall_uid).await.map_err(error_reporter::Report::new).map_err(ServerFnError::new)?;
-
-    Ok(RonEncoded::new(problems))
-}
-
 /// Returns user interaction for a single wall problem
 #[server(
     input = Ron,
@@ -142,12 +79,12 @@ pub(crate) async fn get_problems_for_wall(
 #[tracing::instrument(err(Debug))]
 pub(crate) async fn get_user_interaction(
     wall_uid: models::WallUid,
-    problem_uid: models::ProblemUid,
+    problem: models::Problem,
 ) -> Result<RonEncoded<Option<models::UserInteraction>>, ServerFnError> {
     use crate::server::db::Database;
     use crate::server::db::DatabaseOperationError;
     use leptos::prelude::expect_context;
-    tracing::trace!("Enter");
+    crate::tracing::on_enter!();
 
     #[derive(Debug, derive_more::Error, derive_more::Display, derive_more::From)]
     enum Error {
@@ -157,13 +94,13 @@ pub(crate) async fn get_user_interaction(
         DatabaseOperation(DatabaseOperationError),
     }
 
-    async fn inner(wall_uid: models::WallUid, problem_uid: models::ProblemUid) -> Result<Option<UserInteraction>, Error> {
+    async fn inner(wall_uid: models::WallUid, problem: models::Problem) -> Result<Option<UserInteraction>, Error> {
         let db = expect_context::<Database>();
 
         let user_interaction = db
             .read(|txn| {
                 let user_table = txn.open_table(crate::server::db::current::TABLE_USER)?;
-                let user_interaction = user_table.get((wall_uid, problem_uid))?.map(|guard| guard.value());
+                let user_interaction = user_table.get(&(wall_uid, problem))?.map(|guard| guard.value());
                 Ok(user_interaction)
             })
             .await?;
@@ -171,7 +108,7 @@ pub(crate) async fn get_user_interaction(
         Ok(user_interaction)
     }
 
-    let user_interaction = inner(wall_uid, problem_uid)
+    let user_interaction = inner(wall_uid, problem)
         .await
         .map_err(error_reporter::Report::new)
         .map_err(ServerFnError::new)?;
@@ -185,31 +122,42 @@ pub(crate) async fn get_user_interaction(
     custom = RonEncoded
 )]
 #[tracing::instrument(err(Debug))]
-pub(crate) async fn get_user_interactions(
+pub(crate) async fn get_user_interactions_for_wall(
     wall_uid: models::WallUid,
-) -> Result<RonEncoded<BTreeMap<models::ProblemUid, models::UserInteraction>>, ServerFnError> {
+) -> Result<RonEncoded<BTreeMap<models::Problem, models::UserInteraction>>, ServerFnError> {
     use crate::server::db::Database;
     use crate::server::db::DatabaseOperationError;
     use leptos::prelude::expect_context;
-    tracing::trace!("Enter");
+    use redb::ReadableTable;
+    crate::tracing::on_enter!();
 
     #[derive(Debug, derive_more::Error, derive_more::Display, derive_more::From)]
     enum Error {
         DatabaseOperation(DatabaseOperationError),
     }
 
-    async fn inner(wall_uid: models::WallUid) -> Result<BTreeMap<models::ProblemUid, models::UserInteraction>, Error> {
+    async fn inner(wall_uid: models::WallUid) -> Result<BTreeMap<models::Problem, models::UserInteraction>, Error> {
         let db = expect_context::<Database>();
 
         let user_interactions = db
             .read(|txn| {
                 let user_table = txn.open_table(crate::server::db::current::TABLE_USER)?;
-                let range = user_table.range((wall_uid, models::ProblemUid::min())..=(wall_uid, models::ProblemUid::max()))?;
-                let user_interactions = range
+                let user_interactions = user_table
+                    .iter()?
+                    .filter(|guard| {
+                        guard
+                            .as_ref()
+                            .map(|(key, _val)| {
+                                let (wall_uid, _problem) = key.value();
+                                wall_uid
+                            })
+                            .map(|wall_uid_| wall_uid_ == wall_uid)
+                            .unwrap_or(false)
+                    })
                     .map(|guard| {
                         guard.map(|(_key, val)| {
                             let val = val.value();
-                            (val.problem_uid, val)
+                            (val.problem.clone(), val)
                         })
                     })
                     .collect::<Result<_, _>>()?;
@@ -224,43 +172,6 @@ pub(crate) async fn get_user_interactions(
     Ok(RonEncoded::new(user_interaction))
 }
 
-#[server(
-    input = Ron,
-    output = Ron,
-    custom = RonEncoded
-)]
-#[tracing::instrument(skip_all, err(Debug))]
-pub(crate) async fn get_problem_by_uid(
-    wall_uid: models::WallUid,
-    problem_uid: models::ProblemUid,
-) -> Result<RonEncoded<models::Problem>, ServerFnError> {
-    use crate::server::db::Database;
-    use crate::server::db::DatabaseOperationError;
-    use leptos::prelude::expect_context;
-    tracing::trace!("Enter");
-
-    #[derive(Debug, derive_more::Error, derive_more::Display)]
-    enum Error {
-        #[display("Problem not found: {_0:?}")]
-        NotFound(#[error(not(source))] models::ProblemUid),
-    }
-
-    let db = expect_context::<Database>();
-    let problem = db
-        .read(|txn| {
-            let table = txn.open_table(crate::server::db::current::TABLE_PROBLEMS)?;
-            let problem = table
-                .get((wall_uid, problem_uid))?
-                .ok_or(Error::NotFound(problem_uid))
-                .map_err(DatabaseOperationError::custom)?
-                .value();
-            Ok(problem)
-        })
-        .await?;
-
-    Ok(RonEncoded::new(problem))
-}
-
 /// Inserts or updates today's attempt.
 #[server(
     input = Ron,
@@ -270,14 +181,14 @@ pub(crate) async fn get_problem_by_uid(
 #[tracing::instrument(err(Debug))]
 pub(crate) async fn upsert_todays_attempt(
     wall_uid: models::WallUid,
-    problem_uid: models::ProblemUid,
+    problem: models::Problem,
     attempt: Option<models::Attempt>,
 ) -> Result<RonEncoded<models::UserInteraction>, ServerFnError> {
     use crate::server::db::Database;
     use crate::server::db::DatabaseOperationError;
     use leptos::prelude::expect_context;
 
-    tracing::trace!("Enter");
+    crate::tracing::on_enter!();
 
     #[derive(Debug, Error, Display, From)]
     enum Error {
@@ -287,20 +198,20 @@ pub(crate) async fn upsert_todays_attempt(
         DatabaseOperation(DatabaseOperationError),
     }
 
-    async fn inner(wall_uid: models::WallUid, problem_uid: models::ProblemUid, attempt: Option<models::Attempt>) -> Result<UserInteraction, Error> {
+    async fn inner(wall_uid: models::WallUid, problem: models::Problem, attempt: Option<models::Attempt>) -> Result<UserInteraction, Error> {
         let db = expect_context::<Database>();
 
         let user_interaction = db
             .write(|txn| {
                 let mut user_table = txn.open_table(crate::server::db::current::TABLE_USER)?;
 
-                let key = (wall_uid, problem_uid);
+                let key = (wall_uid, problem.clone());
 
                 // Pop or default
                 let mut user_interaction = user_table
-                    .remove(key)?
+                    .remove(&key)?
                     .map(|guard| guard.value())
-                    .unwrap_or_else(|| models::UserInteraction::new(wall_uid, problem_uid));
+                    .unwrap_or_else(|| models::UserInteraction::new(wall_uid, problem));
 
                 // If the last entry is from today, remove it
                 if let Some(entry) = user_interaction.attempted_on.last_entry() {
@@ -327,7 +238,66 @@ pub(crate) async fn upsert_todays_attempt(
         Ok(user_interaction)
     }
 
-    inner(wall_uid, problem_uid, attempt)
+    inner(wall_uid, problem, attempt)
+        .await
+        .map_err(error_reporter::Report::new)
+        .map_err(ServerFnError::new)
+        .map(RonEncoded::new)
+}
+
+/// Sets is_favorite field for a problem
+#[server(
+    input = Ron,
+    output = Ron,
+    custom = RonEncoded
+)]
+#[tracing::instrument(err(Debug))]
+pub(crate) async fn set_is_favorite(
+    wall_uid: models::WallUid,
+    problem: models::Problem,
+    is_favorite: bool,
+) -> Result<RonEncoded<models::UserInteraction>, ServerFnError> {
+    use crate::server::db::Database;
+    use crate::server::db::DatabaseOperationError;
+    use leptos::prelude::expect_context;
+
+    crate::tracing::on_enter!();
+
+    #[derive(Debug, Error, Display, From)]
+    enum Error {
+        #[display("Wall not found: {_0:?}")]
+        WallNotFound(#[error(not(source))] models::WallUid),
+
+        DatabaseOperation(DatabaseOperationError),
+    }
+
+    async fn inner(wall_uid: models::WallUid, problem: models::Problem, is_favorite: bool) -> Result<UserInteraction, Error> {
+        let db = expect_context::<Database>();
+
+        let user_interaction = db
+            .write(|txn| {
+                let mut user_table = txn.open_table(crate::server::db::current::TABLE_USER)?;
+
+                let key = (wall_uid, problem.clone());
+
+                // Pop or default
+                let mut user_interaction = user_table
+                    .remove(&key)?
+                    .map(|guard| guard.value())
+                    .unwrap_or_else(|| models::UserInteraction::new(wall_uid, problem));
+
+                user_interaction.is_favorite = is_favorite;
+
+                user_table.insert(key, user_interaction.clone())?;
+
+                Ok(user_interaction)
+            })
+            .await?;
+
+        Ok(user_interaction)
+    }
+
+    inner(wall_uid, problem, is_favorite)
         .await
         .map_err(error_reporter::Report::new)
         .map_err(ServerFnError::new)

flake.lock

@@ -10,11 +10,11 @@
         ]
       },
       "locked": {
-        "lastModified": 1741078851,
-        "narHash": "sha256-1Qu/Uu+yPUDhHM2XjTbwQqpSrYhhHu7TpHHrT7UO/0o=",
+        "lastModified": 1746696290,
+        "narHash": "sha256-YokYinNgGIu80OErVMuFoIELhetzb45aWKTiKYNXvWA=",
         "owner": "plul",
         "repo": "basecamp",
-        "rev": "3e4579d8b4400506e5f53069448b3471608b5281",
+        "rev": "108ef2874fd8f934602cda5bfdc0e58a541c6b4a",
         "type": "github"
       },
       "original": {
@@ -25,11 +25,11 @@
     },
     "nixpkgs": {
       "locked": {
-        "lastModified": 1742456341,
-        "narHash": "sha256-yvdnTnROddjHxoQqrakUQWDZSzVchczfsuuMOxg476c=",
+        "lastModified": 1746576598,
+        "narHash": "sha256-FshoQvr6Aor5SnORVvh/ZdJ1Sa2U4ZrIMwKBX5k2wu0=",
         "owner": "NixOS",
         "repo": "nixpkgs",
-        "rev": "7344a3b78128f7b1765dba89060b015fb75431a7",
+        "rev": "b3582c75c7f21ce0b429898980eddbbf05c68e55",
         "type": "github"
       },
       "original": {
@@ -53,11 +53,11 @@
         ]
       },
       "locked": {
-        "lastModified": 1742524367,
-        "narHash": "sha256-KzTwk/5ETJavJZYV1DEWdCx05M4duFCxCpRbQSKWpng=",
+        "lastModified": 1746671794,
+        "narHash": "sha256-V+mpk2frYIEm85iYf+KPDmCGG3zBRAEhbv0E3lHdG2U=",
         "owner": "oxalica",
         "repo": "rust-overlay",
-        "rev": "70bf752d176b2ce07417e346d85486acea9040ef",
+        "rev": "ceec434b8741c66bb8df5db70d7e629a9d9c598f",
         "type": "github"
       },
       "original": {

justfile

@@ -8,6 +8,9 @@ fmt:
     fd --extension=rs --exec-batch leptosfmt
     bc-fmt
 
+fix:
+    bc-fix
+
 serve:
     RUST_BACKTRACE=1 cargo leptos watch -- serve
 

notes.txt

@@ -0,0 +1,16 @@
+# Random selection: Sample from filtered
+[patterns with variations that satisfy filter]
+	[variations]
+
+# Random selection no filter: Sample equally weighted patterns
+[patterns]
+	[random variation within pattern]
+
+Normalize: shift left, and use minimum of mirrored pattern pair
+
+
+# Filter stats:
+patterns: X
+pattern variations: Y
+
+