-
Notifications
You must be signed in to change notification settings - Fork 46
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
Dubious value added of the 'freq' variable #288
Comments
This is intentional behaviour but if many people find this annoying you can report it under this issue and we can reconsider. |
Statistical agencies worldwide have similar standards treating metadata, and metadata in this case is there to avoid unforeseen logical errors when joining or linking data; I think that the freq variable is present when there are similar statistical products or datasets available with the same variables but different frequencies. In that case a joining without frequency adjustment results in a hard to find logical error. The freq variable is the same as the unit variable, you really want to avoid unknowingly divide euros with thousand euros, or multiply annual values in a chain with quarterly values. |
Well, we are all aware. At least I hope so it is the case.
This would assume users are somewhat unaware of what they are doing. It seems to me that implementation of this technique is triumph of form over content. |
@rideofyourlife I have uploaded some WIP code in v4.1 branch. It enables users to make queries the same way as before but adds an additional parameter If you could test this and give some feedback on what you think it would be great! |
I have already laboriously replaced "time" with "TIME_PERIOD" in all my codes, so having "time" back is not as essential now as it had been before the recent change. Despite that, where do I use this |
I'm sorry for the laborious process. In version 4.1 |
Ah, yes: it works. It is just not suggested by R Studio while writing for some reason. |
Many datasets, which contain only one frequency available (like namq_10_gdp, sts_inpr_m etc.), were awarded a new variable "freq". I generally understand the idea behind it, but while working on the package it has only proven to often be an unnecessary step of
%>% select (-freq)
in majority of the code I write.Does anyone else have similar thoughts?
The text was updated successfully, but these errors were encountered: