Problem with Getting_started.ipynb

[hepdata]

Problem with Getting_started.ipynb

[AndreasAlbert]

Hi @rappoccio,

thanks for the report.
Could you check which PyYAML version you are using?
(Run pip show PyYAML in a terminal and copy the output).

Thanks

[rappoccio]

It's 4.1

Name: PyYAML
Version: 4.1
Summary: YAML parser and emitter for Python
Home-page: https://github.com/yaml/pyyaml
Author: Kirill Simonov
Author-email: [email protected]
License: MIT
Location: /opt/local/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/site-packages
Requires: 
Required-by: hepdata-lib

[AndreasAlbert]

Ok, that is the problem. I can reproduce your problem if I install 4.1. I was previously running with 3.12, which worked fine. So if you want to have a quick workaround, you can try downgrading to 3.12.

As a way forward, I would propose to figure out what the right syntax for what we want to do is in 4.1, use that syntax, and update the setup requirements to make sure that at least this version is installed.

[rappoccio]

Aha, thanks for the quick checks! I can dig into this at some point soon if you don't get around to it.

[AndreasAlbert]

@rappoccio If you want to take a look, that would be very welcome! The technical issue here is that we want the yaml package to be able to dump our default_dict just as it would usually be able to dump a normal python dict. I expect this to be reasonably straightforward to fix. Please assign the issue to yourself once you start working on it to keep track of what's going on. Otherwise @clelange or I will look into it.

[rappoccio]

When I get around to it I will put the assignment in.

[clelange]

For which command in the notebook does this error show? I think we can fix it by using either @staticmethod or @classmethod as discussed at https://stackoverflow.com/questions/40701142/typeerror-unbound-method-must-be-called-with-instance-as-first-argument-got-in/40701173
I'll look into it.

[AndreasAlbert]

It happens any time we use yaml.dump to dump our defaultdicts.

[AndreasAlbert]

If we end up switching pyyaml versions, we should revisit #18

[clelange]

Hmm, so the offender seems to be https://github.com/clelange/hepdata_lib/blob/master/hepdata_lib/__init__.py#L18

I've created a new branch (https://github.com/clelange/hepdata_lib/tree/test_yaml_dump) with a couple of additional tests and forcing PyYAML to version 4.1, but I cannot reproduce the error. Can one of you take it from here?

[rappoccio]

I still see the error here. Running out of battery though so can't debug right now haha.

[AndreasAlbert]

Should be fixed in master now. @rappoccio please report if you encounter any more issues.

[rappoccio]

Thanks!

[clelange]

New version 0.2.2 is available on PyPI: https://pypi.org/project/hepdata_lib/