-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
[Bug]: Documentation typo in rtables-scoring-functions section of pages #1118
Comments
sorting happens on the base of extracted values (may it be simply the cell values or something more complex) that I guess were termed scores to imply there are some other possible sorting statistics. If you look at a simple one: > score_occurrences
function(table_row) {
row_counts <- h_row_counts(table_row)
sum(row_counts)
} this extracts all values from a row (that you can reach with the right path in In the following section there is more info on how to create your own scoring function and sorting: https://insightsengineering.github.io/rtables/main/articles/sorting_pruning.html#sorting Note that renaming this set of functions as sorting functions instead of scoring functions would be not precise as the scoring functions only extract values to be sorted. I am open to other suggestions though |
Thanks, I had a closer look at the page and example you provided as well as the reference documentation for The way I found it confusing as a user reading the documentation is that:
Overall, I view these as the functions used as arguments in either Based on the above, I suggest changing the documentation but ideally also the function naming:
So, for example:
Then perhaps the functions themselves could be classed in a way that omits the word However, even just improving documentation for reference slightly would help. At the moment the user has to dig a bit too deep to be used inside of sort_at_path. Hopefully, I understood this well and my comment isn't based on wrong assumptions here. I apologise for using the word |
We will update the docs to reflect this. It was already updated a couple of months ago as it was really unclear how to get it to work with dataRows. For the naming, I am wondering if sort function is better as it invokes the right associations. What do you think @shajoezhu @ayogasekaram @edelarua ? another non-mutually-exclusive possibility would be to have a couple of wrapped sorting functions that only sort the last node of the tree for each column or all (e.g. sort_table). Would it help too @martincadek? |
@Melkiades, I think that updating the documentation will help. I am not sure if I understood the second option. I suppose that from a user perspective using the function |
What happened?
Hello, the package documentation includes a fairly confusing typo which describes
Scoring functions
astable 'sorting'
functions.The typo occurs in the following section of Tern documentation: https://insightsengineering.github.io/tern/latest-tag/reference/index.html#rtables-scoring-functions.
I would be happy to resolve the issue myself, however, I do not know how to directly contribute to the code base.
sessionInfo()
No response
Relevant log output
No response
Code of Conduct
Contribution Guidelines
Security Policy
The text was updated successfully, but these errors were encountered: