Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

No UnbndGenerator #20

Open
slegare2 opened this issue Sep 19, 2018 · 2 comments
Open

No UnbndGenerator #20

slegare2 opened this issue Sep 19, 2018 · 2 comments

Comments

@slegare2
Copy link
Collaborator

Any time I wanted to include an Unbinding interaction, I got the error

File "/home/slegare/ENS/venv_meta_update/lib/python3.6/site-packages/kami-1.2-py3.6.egg/kami/hierarchy.py", line 851, in add_interaction
    nugget, nugget_type = generate_from_interaction(self, interaction)
TypeError: 'NoneType' object is not iterable

I fixed this by adding the following at line 925 of generators.py

elif isinstance(interaction, Unbinding):
        gen = BndGenerator(hierarchy)
        return gen.generate(interaction)

It works since BndGenerator already sets the "test" parameter to False if the object is not an instance of Binding.
I am confused why this bug occured, since Unbindings were working properly in the past ?
If this fix is ok with you, I can push it.

@eugeniashurko
Copy link
Contributor

Hi! I am in the middle of correcting it, sorry was busy a bit. Will push the fix soon. Don't really understand how it got broken, but Unbinding data structure is quite new in general and may be poorly tested

@slegare2
Copy link
Collaborator Author

slegare2 commented Sep 20, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants