Skip to content

Commit 08edeb2

Browse files
authored
Merge pull request #1743 from maths/iss1739
Allow Omega as a synonym for Ohm in units. Fixes #1739
2 parents 59cac8c + 152c2ea commit 08edeb2

3 files changed

Lines changed: 12 additions & 8 deletions

File tree

stack/cas/casstring.units.class.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -91,6 +91,7 @@ class stack_cas_casstring_units {
9191
['Wh', '3600*(kg*m^2)/s^2', 'Wh', 'Watts hours'],
9292
['A', 'A', 'A', 'Ampere'],
9393
['ohm', '(kg*m^2)/(s^3*A^2)', '\Omega', 'ohm'],
94+
['Omega', '(kg*m^2)/(s^3*A^2)', '\Omega', 'ohm'],
9495
['C', 's*A', 'C', 'Coulomb'],
9596
['V', '(kg*m^2)/(s^3*A)', 'V', 'Volt'],
9697
['F', '(s^4*A^2)/(kg*m^2)', 'F', 'Farad'],

tests/ast_filter_998_security_auto_generated_test.php

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -873,6 +873,14 @@ public function test_affected_units(): void {
873873
true
874874
);
875875

876+
$this->expect(
877+
'omega',
878+
'omega',
879+
['unknownUnitsCase'],
880+
false,
881+
true
882+
);
883+
876884
$this->expect(
877885
'partialdiff(x,y,1)',
878886
'partialdiff(x,y,1)',
@@ -2406,14 +2414,6 @@ public function test_non_affected_units(): void {
24062414
false
24072415
);
24082416

2409-
$this->expect(
2410-
'omega',
2411-
'omega',
2412-
[],
2413-
true,
2414-
false
2415-
);
2416-
24172417
$this->expect(
24182418
'omicron',
24192419
'omicron',

tests/fixtures/answertestfixtures.class.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2989,6 +2989,9 @@ class stack_answertest_test_data {
29892989
['Units', '3', '-9.81*m/s^2', 'stackunits(-9.815,m/s^2)', 0, 'ATNumSigFigs_Inaccurate. ATUnits_units_match.', ''],
29902990
['Units', '3', '-9.82*m/s^2', 'stackunits(displaydp(-9.815,3),m/s^2)', 1, 'ATUnits_units_match.', ''],
29912991
['Units', '3', '-9.82*m/s^2', 'stackunits(displaysf(-9.815,4),m/s^2)', 1, 'ATUnits_units_match.', ''],
2992+
['Units', '3', '0.330*kohm', '330*ohm', 1, 'ATUnits_compatible_units (kg*m^2)/(A^2*s^3).', ''],
2993+
['Units', '2', '330*Omega', '330*ohm', 1, 'ATUnits_compatible_units (kg*m^2)/(A^2*s^3).', ''],
2994+
['Units', '2', "330*\u{03A9}", '330*ohm', 1, 'ATUnits_compatible_units (kg*m^2)/(A^2*s^3).', ''],
29922995

29932996
['UnitsStrict', '2', '25*g', '0.025*kg', 0, 'ATUnits_compatible_units kg.', 'Differences from the Units test only'],
29942997
['UnitsStrict', '1', '1*Mg/10^6', '1*N*s^2/(km)', 0, 'ATUnits_compatible_units kg.', ''],

0 commit comments

Comments
 (0)