Skip to content

Conversation

@galou
Copy link

@galou galou commented Jun 29, 2020

No description provided.

@aoloe aoloe merged commit 547d122 into aoloe:master Jun 30, 2020
@aoloe
Copy link
Owner

aoloe commented Jun 30, 2020

thanks for the fixes!

@aoloe
Copy link
Owner

aoloe commented Jun 30, 2020

sorry, the _ was needed. i've re-added it in the next commit.

@galou
Copy link
Author

galou commented Jul 1, 2020

Yes, I added the _ because locals is a reserved word. By the way, how is it possible that set_the_answeris available inmainbut not inget_the_value`? Thanks

@aoloe
Copy link
Owner

aoloe commented Jul 2, 2020

yeah, i've seen it later.
i've now renamed it to outer_locals.

the reason why the injected function is not available inside of the other function, is related to the way pybind11 runs the python code.
i don't have a deep knowledge about it, but i've been told that it's called as if it's inside of a class.

some notes about it are in the README file in the same directory : - )

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants