Skip to content

Conversation

@ebfortin
Copy link
Owner

No description provided.

tmilnthorp and others added 4 commits July 18, 2019 01:34
* Adding benchmark project

* Adding IQuantity tests

* Add memory stats

* Migrate to netcore to fix build error
* Added RatioChangeRate unit

* Fixing issues for RatioChangeRate

- Use of DecimalFractionPerSecond as base unit
- Correction of the documentation
- Adding abbreviation for DecimalFractionPerSecond
* Allowing custom unit conversion functions to be specified

* Registering default base->unit and unit->base conversion functions in UnitConverter. Adding test for custom direct conversions

* Whitespace consistency

* Add some overloads and allowing source/target quantities to be different

* Inline initialization and style update for field

* PR feedback updates

* Regen WRC

* Use IQuantity.Value in test case

* Rename typeparams, remove empty ctor

* Micro optimization of lookup of IQuantity type

It was looked up and wrapped N times

* UnitConverter: Move static field initialization to static ctor

They became a bit unwieldy.
Move fields to top of class.

* Add typed conversion function for same quantity conversion functions

Since this is the most commonly used call path by far, it made sense to handle this case separately.
The advantage is that the callback quantity is strongly typed.

* Use typed conversions in RegisterDefaultConversions

* Header update

* Add xmldoc and minor renaming

* Add test cases for Type parameters

* QuantityParser: Make field readonly

* UnitParser: Add test parsing HowMuch custom unit

* UnitConverter: Echo quantity when converting to same unit, test custom quantity conversions

* Port codegen scripts to C#

The only difference is fixing the header of a file.

* Regen code
@ebfortin ebfortin merged commit ea27c15 into ebfortin:master Jul 20, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants