docs(rust): Fix incorrect column name in LazyFrame.sort doc example

[wsyxbcl]

#15590

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.33%. Comparing base (70b71da) to head (681bae5).
Report is 18 commits behind head on main.

❗ Current head 681bae5 differs from pull request most recent head 9b189c1. Consider uploading reports for the commit 9b189c1 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #15658      +/-   ##
==========================================
- Coverage   81.33%   81.33%   -0.01%     
==========================================
  Files        1374     1374              
  Lines      176471   176471              
  Branches     2544     2544              
==========================================
- Hits       143539   143528      -11     
- Misses      32450    32460      +10     
- Partials      482      483       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[codspeed-hq]

CodSpeed Performance Report

Merging #15658 will not alter performance

_{Comparing wsyxbcl:patch-2 (9b189c1) with main (e218203)}

Summary

✅ 22 untouched benchmarks

[stinodego]

Well spotted! However, The example below is still using 'a', 'b' as columns. Better to just update the preceeding text "sort dataframe by sepal width column" to "Sort DataFrame by a single column". Then it all makes sense.

[wsyxbcl]

Sure, I noticed there are 'sepal width' used elsewhere in the doc, if that's not the consistency we're going to keep, the "a, b" treatment is more clean and clear. Stijn de Gooijer ***@***.***> 于 2024年4月16日周二 上午12:15写道：

…

***@***.**** requested changes on this pull request. Well spotted! However, The example below is still using 'a', 'b' as columns. Better to just update the preceeding text "sort dataframe by sepal width column" to "Sort DataFrame by a single column". — Reply to this email directly, view it on GitHub <#15658 (review)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADDYEBTWXK42PKQCUBLOJN3Y5P4KFAVCNFSM6AAAAABGHBHUNCVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAMBRGUZTQOBTGU> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[stinodego]

You're right. Then maybe update the last example to use sepal.width and petal.width or something, instead of a and b.

[wsyxbcl]

You're right. Then maybe update the last example to use sepal.width and petal.width or something, instead of a and b.

Sure, then we stick to the iris dataset as before

[stinodego]

I saw some inconsistency in using sepal.width vs sepal_width so I updated all references to use the same column names as the small dataset we have in the docs folder.

This can be merged, thanks 👍