expectedparrot / edsl

Design, conduct and analyze results of AI-powered surveys and experiments. Simulate social science and market research with large numbers of AI agents and LLMs.
https://docs.expectedparrot.com
MIT License
97 stars 14 forks source link

order_by seems to print a dictionary ? #246

Open johnjosephhorton opened 2 months ago

johnjosephhorton commented 2 months ago
Screen Shot 2024-04-11 at 10 11 26 AM
rbyh commented 2 months ago

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 commented 2 months ago

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 commented 2 months ago

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.