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

etable.py needs from leabra import env #2

Closed
thazy opened this issue Sep 20, 2019 · 2 comments
Closed

etable.py needs from leabra import env #2

thazy opened this issue Sep 20, 2019 · 2 comments

Comments

@thazy
Copy link

thazy commented Sep 20, 2019

After adding a call to self.ConfigPats() in the ra25.py demo project (from .../leabra/examples/ra25/) entering into the function call patgen.PermutedBinaryRows(dt.Cols[1], 6, 1, 0) (~line 766) ends up in etable.py where env is a missing variable. When I add the from' line locally and then did .../leabra/python/make install' (Important: do not do a make since that appears to restore the env-less version) it appears to fix the problem, only to seg fault further downstream (see .../emer/leabra/ Issue #1).

@rcoreilly
Copy link
Member

this is a bug in gopy, which I'll have to fix at some point soon. I was able to fix this specific issue by moving the order of etable in front of emergent in the gopy command in leabra/python/Makefile, but then another one cropped up involving agg. it just needs smarter logic on these imports.

@rcoreilly
Copy link
Member

This was fixed in pr198 of gopy which has yet to be applied. In case anyone else is looking at this, you need to do this in gopy to get the update:

$ cd ~/go/src/github.com/go-python/gopy  # use $GOPATH instead of ~/go if somewhere else
$ git fetch origin pull/198/head:pr198  # this gets the pull-request #198, into branch pr198
$ git checkout pr198  # switch to that branch
$ go install    # do go get -u ./... if this fails and try again -- installs gopy exe in ~go/bin

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

No branches or pull requests

2 participants