-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Decimal vs double #3264
Comments
@CraigElder did you end up getting a tlog for this one? |
can we assume that if the value is larger that decimal.maxvalue then we can ignore conversion issues and cast it directly to double ? |
The only remaining question is the first one: can we assume that if the value is larger that decimal.maxvalue then we can ignore conversion issues and cast it directly to double. |
I've seen an issue where a cast from double to decimal causes an exception. Need to fix mavparam to remove all decimals
An int32 can be 100% represented by a double. So replace decimal with double. Only issue is with float rounding. So needs to be considered
Ie decimal.maxvalue < float.maxvalue
The text was updated successfully, but these errors were encountered: