search

aks2203 / deep-thinking

A centralized place for deep thinking code and experiments
MIT License
73 stars 12 forks source link

Error when running make_schoop.py #4

Closed Maosef closed 2 years ago

Maosef commented 2 years ago

Context: I'm trying to produce plots for prefix sums. I successfully created a pivot table (after installing tabulate) by running

python deepthinking/data_analysis/make_table.py results/output_default/

However, running the command python deepthinking/data_analysis/make_schoop.py results/output_default/ produces the following error:

image

It looks like there is a missing column test_acc_sem, not sure why the pivot table code isn't adding that.

aks2203 commented 2 years ago

Hi, sem refers to standard error and is not computable with only one trial (see the column count_count). That's why you're seeing this, but I will fix it so that if if can't compute sem it won't try.

aks2203 commented 2 years ago

See the commit: https://github.com/aks2203/deep-thinking/commit/9246866cf5e9a0bebe6a02296f3dc20190b3dd77

I think the issue is fixed. If you can confirm, then we'll go ahead and close this issue.

Maosef commented 2 years ago

I wasn't able to checkout the commit (fatal: reference is not a tree: 9246866cf5e9a0bebe6a02296f3dc20190b3dd77), but implementing the change does fix things, thanks!

aks2203 commented 2 years ago

From your local machine you may need to run $ git fetch or $ git pull to get that commit locally before trying to check it out. Let me know if that helps.

Maosef commented 2 years ago

I confirm the change fixes things!