Migrate from temperaments package to sonic-weave/Temperament

Fix circle step pattern display with multiple periods. Remove support for non-equave constraints.
xenharmonic-devs · Jul 10, 2024 · 0aeaea0 · 0aeaea0
1 parent e32c473
commit 0aeaea0
Show file tree

Hide file tree

Showing 10 changed files with 446 additions and 975 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -22,9 +22,8 @@
     "moment-of-symmetry": "^0.8.2",
     "pinia": "^2.1.7",
     "qs": "^6.12.0",
-    "sonic-weave": "0.9.0",
+    "sonic-weave": "^0.10.1",
     "sw-synth": "^0.1.0",
-    "temperaments": "^0.5.3",
     "values.js": "^2.1.1",
     "vue": "^3.3.4",
     "vue-router": "^4.3.0",

diff --git a/src/__tests__/tempering.spec.ts b/src/__tests__/tempering.spec.ts
diff --git a/src/components/PeriodCircle.vue b/src/components/PeriodCircle.vue
@@ -109,7 +109,8 @@ const mosLabel = computed(() => {
       else if (step === M) countM++
       else if (step === L) countL++
     }
-    return `${countL}L ${countM}M ${countS}s`
+    const n = props.numPeriods
+    return `${n * countL}L ${n * countM}M ${n * countS}s`
   }
   // There are degenerate edgecases with step variety = 2. Show nothing to avoid implying MOS.
   return ''