edsl icon indicating copy to clipboard operation
edsl copied to clipboard

`select("comment.*")` should omit free text (nan) columns by default

Open rbyh opened this issue 1 month ago • 5 comments

rbyh avatar Nov 10 '25 15:11 rbyh

@rbyh Can you give more details about this? select("comment.*") what is this?

Cprakhar avatar Nov 27 '25 14:11 Cprakhar

This will give you all the columns of results that have a "comment" prefix - there is a comment field for each question. Each question other than free text includes a prompt for the agent/model to provide its reasoning for the answer. You can turn off comments if not desired.

rbyh avatar Nov 27 '25 15:11 rbyh

@rbyh Can you assign this issue to me? I want to work on it. In the issue title you wrote by default, which means user can change default by include_free_text_comments=True like result.select("comment.*", include_free_text_comments=True)

Cprakhar avatar Nov 27 '25 17:11 Cprakhar

What about other wildcard types "*.*" or "comment.some_question"?

Cprakhar avatar Nov 27 '25 18:11 Cprakhar

I actually added more general wild-card support recently. I wouldn't jump into this one quite yet - I'm actually not sure why comments are being including in results for free text.

johnjosephhorton avatar Nov 27 '25 18:11 johnjosephhorton