-
Notifications
You must be signed in to change notification settings - Fork 405
QuantityValue implemented as a fractional number 🐲 #1544
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.
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
lipchev
wants to merge
67
commits into
angularsen:master
Choose a base branch
from
lipchev:fractional-quantity-value
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
67 commits
Select commit
Hold shift + click to select a range
3c61dd6
- QuantityValue implemented as a fractional number
lipchev e421a1b
adding the git-ignored files
lipchev 53f00cd
changing the CultureInfo? parameters back to IFormatProvder?
lipchev e339d87
- `UnitAbbreviationsCahe` and `UnitParser`: replaced the IReadOnlyCol…
lipchev d3092bc
including the changes from https://github.com/angularsen/UnitsNet/pul…
lipchev 5693259
Completed the QuantityFormatter test coverage (and some cosmetic chan…
lipchev b027df2
UnitKey: making the constructor internal and renaming the properties
lipchev e23973a
removing the QuantityFormatter overloads and re-organizing the extens…
lipchev fe0a4c9
- updating the implicit usings and cleanup some of the using statements
lipchev 56b1df9
adding a few more tests
lipchev da9a3e9
removing some unnecessary using statements and replacing the target f…
lipchev 345ae06
renaming the default quantity provider
lipchev 3c84a32
removing Quantity.FromQuantityInfo
lipchev 5907e18
avoiding the ToArray call when initializing the quantity info using a…
lipchev 82edf5c
Merge remote-tracking branch 'upstream\master' into fractional-quanti…
lipchev b07cab1
Merge remote-tracking branch 'upstream\master' into fractional-quanti…
lipchev e27cc5b
removing the remnents of the IQuantityInfo<TUnit> interface and the u…
lipchev faa0c1e
removed the unit-not-found comment
lipchev eb22805
merging the removal of the EnumUtils (i'm keeping it for now)
lipchev ece337d
merging the AoT compatibility: https://github.com/angularsen/UnitsNet…
lipchev cdf2016
adding the missing TryGetUnitFromAbbreviation overload and the tests
lipchev b1130f4
merging from upstream: ✨Add more Gray units (#1568)
lipchev f425ebe
removed the EnumUtils and replacing the usages with the EnumHelper
lipchev c53e7e9
bumping the pre-release version to 6.0.0-pre017 (staying ahead of the…
lipchev e5664e7
merging from upstream (keeping ahead of the official nugget version: …
lipchev e43068f
cleanup some namespaces and other cosmetic changes
lipchev 35619ee
merging from upstream and bumping the nugget versions (keeping ahead …
lipchev ffc88de
splitting the IQuantity interface definitions into separate files
lipchev f7118a4
- renamed IQuantityInstance to IQuantityOfType
lipchev 192ecae
making the GenericMathExtensions class obsolete and replacing the rel…
lipchev 75fe28c
remove the GenerateArithmetic from the Temperature.json
lipchev fb01d4f
- re-formatting the QuantityInfo and making the IQuantityInfo interna…
lipchev db8ee3e
cleanup and reformat the UnitInfo/IUnitDefinition
lipchev b8877e1
merging from upstream: Add prefix milli to torr unit (#1588)
lipchev 8c31f4e
cosmetic changes
lipchev f94a4d0
refactored test: From_ValueAndUnit_ReturnsQuantityWithSameValueAndUnit
lipchev 0133647
renamed the UnitsNetSetup.QuantityInfoLookup
lipchev 338da80
Merge remote-tracking branch 'upstream\master' into fractional-quanti…
lipchev 09fd680
synchronizing the changes in UnitKey
lipchev 8634470
merged from upstream (ignoring the ToUnitUntyped bits)
lipchev 6e229c7
- Constraint the Abs extension method to ILinearQuantity<TQuantity>
lipchev 927b7e2
merged from upstream
lipchev e569dfa
updating the nugget packages
lipchev 6d430a6
refactored the [Obsolete] IQuantity methods as extensions
lipchev f7fc596
exposing the operator IDivisionOperators<TSelf, TSelf, QuantityValue>
lipchev 1538a52
bump the nugget version to 6.0.0-pre019
lipchev 14ee96e
making the TQuantity Create(double, UnitKey) method public (IQuantity…
lipchev 48d50a9
removing the Dimensions instance property
lipchev 2724c42
merged from upstream
lipchev 3b3287d
removed the Dimensions from the HowMuch
lipchev ab3402b
completing the implementation of the HowMuch in the Configuration sample
lipchev f549040
merged from upstream
lipchev d13e1fe
Merge remote-tracking branch 'origin/master' into fractional-quantity…
angularsen 7128b1b
Samples/Nuget.config: Use artifcats folder to access all locally buil…
angularsen 896f765
samples/UnitConverter.Wpf: Use double example values
angularsen a7683b8
samples: Add 'Official' solution configuration, remove all but AnyCPU
angularsen af6a79f
Revert "samples/UnitConverter.Wpf: Use double example values"
angularsen ff48212
samples: ⬆️UnitsNet 5.75.0
angularsen 85356b4
♻️Rename to 'expected' for some test cases
angularsen 9263c9e
♻️Rename to WithConversionFactorFrom/ToBase
angularsen 697f08d
⚡Use GeneratedRegex in ExpressionEvaluator
angularsen e1d00f5
🐛Fix fraction parsing on non-English system, add FractionHelper.TryPa…
angularsen 59ae553
♻️Specify invariant culture for all int parsing
angularsen 6c5648e
♻️Add parameter names for ConversionExpression ctor arguments
angularsen 1c9459a
♻️Fix whitespace in generated quantities
angularsen 0623de4
♻️Fix inconsistent ordering of UnitRelations.json, use OrdinalIgnoreCase
angularsen 46848ce
♻️Use named arguments for significantDigits
angularsen File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,4 +1,4 @@ | ||
using System; | ||
using System; | ||
using CodeGen.JsonTypes; | ||
|
||
namespace CodeGen.Generators.NanoFrameworkGen | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SystemTextJson is candidate for separate PR
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Never mind, let's just review and merge everything