hydraulic unit fix ups

[BenTWhite]

1. fixed the length_short formatter to use mm and inches
2. added an infiltration_intensity formatter for rainfall intensity
3. added a side slope formatter which uses the inverted slope unit - for the side slopes trapezoidal channels along with some coming attributes associated with storage

[BenTWhite]

@ColinKerr @naveedkhan8067 @diegoalexdiaz

the IModelNativeTests are failing - not sure what is happening. Let me know if it is something on my side.

[naveedkhan8067]

@ColinKerr @naveedkhan8067 @diegoalexdiaz

the IModelNativeTests are failing - not sure what is happening. Let me know if it is something on my side.

@BenTWhite , its not because of your changes. There is a PR on imodel-native side that got merge and its relevant changes on iTwin.js side are still waiting. As soon as that PR merge these tests should pass.

[naveedkhan8067]

/azp run iModelNativeTests

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[naveedkhan8067]

@ColinKerr @naveedkhan8067 @diegoalexdiaz
the IModelNativeTests are failing - not sure what is happening. Let me know if it is something on my side.

@BenTWhite , its not because of your changes. There is a PR on imodel-native side that got merge and its relevant changes on iTwin.js side are still waiting. As soon as that PR merge these tests should pass.

@BenTWhite, it should be working now.

[BenTWhite]

@naveedkhan8067

Just a heads up - am getting this failure message:

src/BriefcaseManager.ts(439,19): error TS2554: Expected 2 arguments, but got 1.

[naveedkhan8067]

/azp run iModelNativeTests

[azure-pipelines]

Azure Pipelines successfully started running 1 pipeline(s).

[naveedkhan8067]

Just a heads up

Seems like iTwin.js build was broken for some time due to another issue as well. I have retriggered the build and it has passed now.