-
Notifications
You must be signed in to change notification settings - Fork 30
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
Error - Potentially on conversion of fields while fetching. #32
Comments
When the paidAmt field is removed from the query, then the results are delivered as expected. |
I changed the paidAmt field to datatype money, as well as numeric, and still am getting the same error. |
This has been covered in #22 and #26. The underlying package rClr doesn't support these datatypes. This is easily fixed with a selectQry <- "SELECT TOP 10 claimType, transType, CAST(paidAmt AS float) FROM tmpRx_01" |
Agreed that casting would fix this. There are 3 solutions besides a manual cast
I've tested the first two successfully for decimal and money, and emailed one of the contributors to see how to get this reviewed for approval and released. I also am trying to figure out how to modify rClr as well, and work with that package for potentially adding other R functionality in the future. |
If you want to get anything added to this package just fork the package, create a new branch and submit a pull request :) Would be great to have a better solution to this! |
Thanks, I just forked the package, uploaded a small tweak to
SqlDataHelper.cs, and submitted a pull request.
Since this is my first time forking and submitting a pull request, please
let me know if I need to do anything else for the workaround to be included
in the project.
…On Thu, Jan 4, 2018 at 7:44 PM, Ruaridh Williamson ***@***.*** > wrote:
If you want to get anything added to this package just fork the package,
create a new branch and submit a pull request
<https://opensource.guide/how-to-contribute/#opening-a-pull-request> :)
Would be great to have a better solution to this!
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#32 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AhQViogQUBBXTN0xwVpxa735D3cFdpZpks5tHZqNgaJpZM4QGNq4>
.
|
I am assuming that it is trying to convert the fetched fields into another data-type, rather than the data type per the database. In the code below, paidAmt is of data type decimal(19,2).
R Code:
Error:
The text was updated successfully, but these errors were encountered: