ContextLab / quail

A python toolbox for analyzing and plotting free recall data
http://cdl-quail.readthedocs.io/en/latest/
MIT License
20 stars 10 forks source link

Unable to run example from README #76

Closed RichardLitt closed 6 years ago

RichardLitt commented 6 years ago

The example in the README errored.

import quail

# presented words
presented_words = [['cat', 'bat', 'hat', 'goat'],['zoo', 'animal', 'zebra', 'horse']]

# recalled words
recalled_words = [['bat', 'cat', 'goat', 'hat'],['animal', 'horse', 'zoo']]

# create egg
egg = quail.Egg(pres=presented_words, rec=recalled_words)

#load data
egg = quail.load_example_data()

#analysis
analyzed_data = quail.analyze(egg, analysis='accuracy', listgroup=['average']*16)

I also tried with the second egg commented out. Should an argument be passed to load_example_data()? Would it be possible to provide an egg in the example which could be used to generate graphs as in the examples? Am I missing something?

Discovered as part of https://github.com/openjournals/joss-reviews/issues/424.

paxtonfitzpatrick commented 6 years ago

@RichardLitt should be fixed now

RichardLitt commented 6 years ago

Thank you. I am now able to run it.

The graph that appears is different in style from what is in the README. Is this expected?

figure_1

paxtonfitzpatrick commented 6 years ago

Yes, that's expected for now. I'll update the graphs that appear in the README to match the output shortly

andrewheusser commented 6 years ago

@RichardLitt the figures are now updated to reflect the output of the README code (e5c441547faed527b054522271e7029ef4470c94)

cc: @paxtonfitzpatrick

RichardLitt commented 6 years ago

The new figures don't look the same as the one I posted above - no background, for instance. Do you have special settings for your output graphs?

andrewheusser commented 6 years ago

Ah! No, I think it stems from the fact that seaborn, a package that we rely on for plotting, updated their default figure styles. I have an older version and you must have the newest. I'll update the figures to reflect the latest version of seaborn.

andrewheusser commented 6 years ago

ok! figures are updated :)