Looky1173 / launchpad-bugs-migration2

0 stars 0 forks source link

[BUG n°1745490] DieselConsumptionTab does not accept gal/h or l/h asa UoM #1979

Open Looky1173 opened 6 years ago

Looky1173 commented 6 years ago

Imported from https://bugs.launchpad.net/bugs/1745490

Property Value
Reported by Gerry (gerry-s-u)
Date reported Thu, 25 Jan 2018 21:42:17 GMT
Tags dieselconsumptiontab

As specified in the manual the default UoM for the DieselConsumptionTab, found in the ORTSDieselEngines block, is gal/kwh. That may be the technically correct term for a diesel engine running a electrical generating plant, however, it is an unfriendly term for most users. I have tried other UoM, gal/h for instance -- the OR log shows the following warnings: "Warning: Found a suffix 'gal/h' which could not be parsed as a Any unit in F:\2TSF\ShermanHill\trains\trainset\SLI_UNION_PACIFIC\openrails\UP_SD70ACE_8367.eng:line 25

Warning: Found a suffix 'gal/h' which could not be parsed as a Any unit in F:\2TSF\ShermanHill\trains\trainset\DWupSD70M\openrails\UP5013.eng:line 28" ***see attached log

All the engine files I've run across (NorthAmerican) list fuel consumption in gal/h, the metric equivalent being l/h.

To make OR more user friendly (specifically the ORTSDieselEngine block) the DieselConsumptionTab should accept gal/h and l/h as optional UoM.

Any conversion to gal/kwh should be done by OR, not the user. My hope is that this presents no undue problems to change in the code. Thank You, Gerry Storey

Looky1173 commented 6 years ago

Imported from https://bugs.launchpad.net/or/+bug/1745490/comments/1

Property Value
Posted by Gerry (gerry-s-u)
Date posted Thu, 25 Jan 2018 21:42:17 GMT
Looky1173 commented 6 years ago

Imported from https://bugs.launchpad.net/or/+bug/1745490/comments/2

Property Value
Posted by Peter Newell (coals-to-newcastle)
Date posted Fri, 26 Jan 2018 03:01:55 GMT

The actual UoM is l/rpm. So it is suggested that use of gal/h or l/h is not appropriate.

It appears that the manual has the incorrect UoM in it. Manual Corrected