CLDR-15216 Add light-speed

unicode-org · May 14, 2024 · 6bb2d48 · 6bb2d48
1 parent 9a667f1
commit 6bb2d48
Show file tree

Hide file tree

Showing 6 changed files with 44 additions and 5 deletions.
diff --git a/common/main/en.xml b/common/main/en.xml
@@ -7514,6 +7514,11 @@ annotations.
 				<unitPattern count="one">{0} knot</unitPattern>
 				<unitPattern count="other">{0} knots</unitPattern>
 			</unit>
+			<unit type="speed-light-speed">
+				<displayName>light</displayName>
+				<unitPattern count="other">{0} light</unitPattern>
+				<unitPattern count="other">{0} light</unitPattern>
+			</unit>
 			<unit type="speed-beaufort">
 				<displayName>Beaufort</displayName>
 				<unitPattern count="one">Beaufort {0}</unitPattern>
@@ -8593,6 +8598,11 @@ annotations.
 				<unitPattern count="one">{0} kn</unitPattern>
 				<unitPattern count="other">{0} kn</unitPattern>
 			</unit>
+			<unit type="speed-light-speed">
+				<displayName>light</displayName>
+				<unitPattern count="other">{0} light</unitPattern>
+				<unitPattern count="other">{0} light</unitPattern>
+			</unit>
 			<unit type="speed-beaufort">
 				<displayName>Bft</displayName>
 				<unitPattern count="one">B {0}</unitPattern>
@@ -9669,7 +9679,12 @@ annotations.
 				<unitPattern count="one">{0}kn</unitPattern>
 				<unitPattern count="other">{0}kn</unitPattern>
 			</unit>
-			<unit type="speed-beaufort">
+			<unit type="speed-light-speed">
+				<displayName>light</displayName>
+				<unitPattern count="other">{0}light</unitPattern>
+				<unitPattern count="other">{0}light</unitPattern>
+			</unit>
+ 			<unit type="speed-beaufort">
 				<displayName>Bft</displayName>
 				<unitPattern count="one">B{0}</unitPattern>
 				<unitPattern count="other">B{0}</unitPattern>

diff --git a/common/main/root.xml b/common/main/root.xml
@@ -5615,6 +5615,10 @@ Warnings: All cp values have U+FE0F characters removed. See /annotationsDerived/
 				<displayName>kn</displayName>
 				<unitPattern count="other">{0} kn</unitPattern>
 			</unit>
+			<unit type="speed-light-speed">
+				<displayName>light</displayName>
+				<unitPattern count="other">{0} light</unitPattern>
+			</unit>
 			<unit type="speed-beaufort">
 				<displayName>Bft</displayName>
 				<unitPattern count="other">B {0}</unitPattern>

diff --git a/common/supplemental/units.xml b/common/supplemental/units.xml
@@ -348,8 +348,8 @@ For terms of use, see http://www.unicode.org/copyright.html
 
         <!-- speed -->
         <convertUnit source='knot' baseUnit='meter-per-second' factor='1852/3600' systems="ussystem uksystem si_acceptable"/>
+        <convertUnit source='light-speed' baseUnit='meter-per-second' factor='speed_of_light_meters_per_second' systems="ussystem uksystem si_acceptable"/>
         <convertUnit source='beaufort' baseUnit='meter-per-second' special='beaufort' systems="metric_adjacent"/>
-
 
         <!-- magnetic-induction -->
         <convertUnit source='tesla' baseUnit='kilogram-per-square-second-ampere' systems='si metric prefixable'/>

diff --git a/common/validity/unit.xml b/common/validity/unit.xml
@@ -241,6 +241,7 @@ For terms of use, see http://www.unicode.org/copyright.html
 			volume-koku
 			mass-fun
 			duration-night
+			speed-light-speed
 		</id>
 		<id type='unit' idStatus='deprecated'>
 			acceleration-meter-per-second-squared

diff --git a/tools/cldr-code/src/main/java/org/unicode/cldr/util/DtdData.java b/tools/cldr-code/src/main/java/org/unicode/cldr/util/DtdData.java
@@ -1869,6 +1869,7 @@ private static final class UnitOrderHolder {
                                         "volume-sai",
                                         "volume-to-jp",
                                         "volume-koku",
+                                        "speed-light-speed",
                                         "mass-fun",
                                         "duration-night"))
                         .freeze();

diff --git a/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestUnits.java b/tools/cldr-code/src/test/java/org/unicode/cldr/unittest/TestUnits.java
@@ -3882,7 +3882,7 @@ public void testUnitSystems() {
                     || cldrSystems.contains(UnitSystem.si) && !nistSystems.contains(UnitSystem.si)
                     || cldrSystems.contains(UnitSystem.si_acceptable)
                             && !nistSystems.contains(UnitSystem.si_acceptable)) {
-                if (unit.equals("100-kilometer")) {
+                if (unit.equals("100-kilometer") || unit.equals("light-speed")) {
                     continue;
                 }
                 fails.add(
@@ -4459,14 +4459,32 @@ public void testFactorsInUnits() {
             {"item-per-1e9-item", "item-per-item", "1/1000000000"},
             {"item-per-1e9", "item", "1/1000000000"},
         };
+        checkConversionToBase(tests);
+    }
+
+    public void checkConversionToBase(String[][] tests) {
+        int count = 0;
         for (String[] test : tests) {
+            ++count;
             Output<String> metric = new Output<>();
             final String inputUnit = test[0];
             final String expectedTarget = test[1];
             final Rational expectedFactor = Rational.of(test[2]);
             ConversionInfo unitId1 = converter.parseUnitId(inputUnit, metric, DEBUG);
-            assertEquals(inputUnit, expectedTarget, metric.value);
-            assertEquals(inputUnit, expectedFactor, unitId1.factor);
+            assertEquals(count + ")\t" + inputUnit + " to base", expectedTarget, metric.value);
+            assertEquals(count + ")\t" + inputUnit + " factor", expectedFactor, unitId1.factor);
         }
     }
+
+    public void testLightSpeed() {
+        String[][] tests = {
+            {"light-speed", "meter-per-second", "299792458"},
+            {"light-speed-second", "meter-second-per-second", "299792458"},
+            {"light-speed-minute", "meter-second-per-second", "299792458*60"},
+            {"light-speed-hour", "meter-second-per-second", "299792458*3600"},
+            {"light-speed-day", "meter-second-per-second", "299792458*86400"},
+            {"light-speed-week", "meter-second-per-second", "299792458*604800"},
+        };
+        checkConversionToBase(tests);
+    }
 }