Skip to content

Add Torr Pressure Conversion #2380

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: main
Choose a base branch
from
Open

Add Torr Pressure Conversion #2380

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
97cf8a4
Refactor KeyboardShortcutManager.cs for improved readability and main…
JacobPoteet Feb 10, 2024
d197d69
Add Pressure_Torr as a new unit constant in UnitConverterDataConstant…
JacobPoteet Feb 10, 2024
79b2bf1
Add Torr unit to pressure units in UnitConverterDataLoader.cpp
JacobPoteet Feb 10, 2024
e5dd948
Add Torr as a unit abbreviation for Pressure
JacobPoteet Feb 10, 2024
d0e3e45
Add Torr as a measurement unit for Pressure.
JacobPoteet Feb 10, 2024
aecb9d3
Add conversion data for "Atmospheres-Torr" in Test.resw
JacobPoteet Feb 10, 2024
f5439bf
Refactor UnitConverterDataLoader.cpp to correctly add pressure units
JacobPoteet Feb 10, 2024
b57154f
Revert unrelated optimization
JacobPoteet Feb 10, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion src/CalcViewModel/DataLoaders/UnitConverterDataConstants.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,8 @@ namespace CalculatorApp
Energy_Kilowatthour = UnitStart + 166,
Data_Nibble = UnitStart + 167,
Length_Angstrom = UnitStart + 168,
UnitEnd = Length_Angstrom
Pressure_Torr = UnitStart + 169,
UnitEnd = Pressure_Torr
};
}
}
5 changes: 4 additions & 1 deletion src/CalcViewModel/DataLoaders/UnitConverterDataLoader.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -758,6 +758,8 @@ void UnitConverterDataLoader::GetUnits(_In_ unordered_map<ViewMode, vector<Order
OrderedUnit{ UnitConverterUnits::Pressure_Pascal, GetLocalizedStringName(L"UnitName_Pascal"), GetLocalizedStringName(L"UnitAbbreviation_Pascal"), 5 });
pressureUnits.push_back(OrderedUnit{
UnitConverterUnits::Pressure_PSI, GetLocalizedStringName(L"UnitName_PSI"), GetLocalizedStringName(L"UnitAbbreviation_PSI"), 6, false, false, false });
pressureUnits.push_back(OrderedUnit{
UnitConverterUnits::Pressure_Torr, GetLocalizedStringName(L"UnitName_Torr"), GetLocalizedStringName(L"UnitAbbreviation_Torr"), 7});
unitMap.emplace(ViewMode::Pressure, pressureUnits);

vector<OrderedUnit> angleUnits;
Expand Down Expand Up @@ -947,7 +949,8 @@ void UnitConverterDataLoader::GetConversionData(_In_ unordered_map<ViewMode, uno
{ ViewMode::Pressure, UnitConverterUnits::Pressure_KiloPascal, 0.0098692326671601 },
{ ViewMode::Pressure, UnitConverterUnits::Pressure_MillimeterOfMercury, 0.0013155687145324 },
{ ViewMode::Pressure, UnitConverterUnits::Pressure_Pascal, 9.869232667160128e-6 },
{ ViewMode::Pressure, UnitConverterUnits::Pressure_PSI, 0.068045961016531 } };
{ ViewMode::Pressure, UnitConverterUnits::Pressure_PSI, 0.068045961016531 },
{ ViewMode::Pressure, UnitConverterUnits::Pressure_Torr, 0.00131578947368 } };

// Populate the hash map and return;
for (UnitData unitdata : unitDataList)
Expand Down
8 changes: 8 additions & 0 deletions src/Calculator/Resources/en-US/Resources.resw
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2361,6 +2361,10 @@
<value>psi</value>
<comment>An abbreviation for a measurement unit of Pressure</comment>
</data>
<data name="UnitAbbreviation_Torr" xml:space="preserve">
<value>Torr</value>
<comment>An abbreviation for a measurement unit of Pressure</comment>
</data>
<data name="UnitAbbreviation_Centigram" xml:space="preserve">
<value>cg</value>
<comment>An abbreviation for a measurement unit of weight</comment>
Expand Down Expand Up @@ -2453,6 +2457,10 @@
<value>Pounds per square inch</value>
<comment>A measurement unit for Pressure.</comment>
</data>
<data name="UnitName_Torr" xml:space="preserve">
<value>Torr</value>
<comment>A measurement unit for Pressure.</comment>
</data>
<data name="UnitName_Centigram" xml:space="preserve">
<value>Centigrams</value>
<comment>A measurement unit for weight. (Please choose the most appropriate plural form to fit any number between 0 and 999,999,999,999,999)</comment>
Expand Down
3 changes: 3 additions & 0 deletions src/CalculatorUnitTests/Test.resw
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -576,6 +576,9 @@
<data name="Atmospheres-Pounds per square inch">
<value>0.068045961016531</value>
</data>
<data name="Atmospheres-Torr">
<value>0.00131578947368</value>
</data>
<data name="Liters-Cubic meters-2" xml:space="preserve">
<value>0.002</value>
</data>
Expand Down