edsl icon indicating copy to clipboard operation
edsl copied to clipboard

order_by seems to print a dictionary ?

Open johnjosephhorton opened this issue 1 year ago • 3 comments

Screen Shot 2024-04-11 at 10 11 26 AM

johnjosephhorton avatar Apr 11 '24 14:04 johnjosephhorton

sort_by() is a method of results but order_by() is not -- did you mean sort_by, and do you think we also need order_by?

https://github.com/expectedparrot/edsl/blob/main/edsl/results/Results.py#L536

rbyh avatar Apr 11 '24 14:04 rbyh

If you think people will try order_by(), should we have it show a message to use sort_by()?

One drawback of the current version of sort_by() is that you have to append it for each column that you want to sort by.

rbyh avatar Apr 11 '24 14:04 rbyh

I think order_by() should refer to the order of the columns in the table. But this could also be handled by the select() method.

rbyh avatar Apr 11 '24 14:04 rbyh