-
Notifications
You must be signed in to change notification settings - Fork 104
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
Removing Scalars from non-Scalar Calculations #9382
Removing Scalars from non-Scalar Calculations #9382
Conversation
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.
@derekagorhom it is only partly working well.
Please can you get a scalar and the try to use it. I used the survey and saved mean(yield) into mean_yield as a scalar. Then tried a calculation of yield * mean_yield.
It says it doesn't recognise mean_yield. It used to do that fine.
@rdstern I have made the fix which prevented the check scalar options from producing results |
@derekagorhom you have solved that problem, but seem to have created some more. You may need help, because it looks to be rather a mess now: Here is what I did: |
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.
@derekagorhom that's a relief. It now seems to be working. Many thanks.
@N-thony can you test as well as check the code. As a presumably separate issue I note that the number of significan figures does not apply to the scalars. They are displayed here with 13 decimals in the data frame metadata. Is this easy to change?
Fixes #9376
This PR should prevent the issue of the scalars function appearing when the user is not using the scalar option.
@rdstern , @N-thony can you review this code
Thanks