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

Make generating c302 from PyOpenWorm work in the OpenWorm Docker image #266

Open
slarson opened this issue Nov 22, 2015 · 5 comments
Open

Comments

@slarson
Copy link
Member

slarson commented Nov 22, 2015

We currently have a start on a docker image that contains PyOpenWorm.

Inside that image is the dev version of the master_openworm.py script

The first step of this script is to generate c302 from the latest contents of PyOpenWorm.

We have previously made a few steps towards accomplishing this:

We need to consolidate progress on these from a code perspective, and move on to making this runnable by modifying the dev version of the master_openworm.py script and committing a new version of the docker image that has this working correctly.

@pgleeson
Copy link
Member

As mentioned in the issues above, #177 is the key to updating c302 to use PyOpenWorm...

@perfortnight
Copy link

November 29, 2015 9:43 AM

Conversation specifying more details about this ticket.

@slarson
Copy link
Member Author

slarson commented Jan 18, 2016

OK now that #177 is done, let's see if we can make more progress. @perfortnight ; any updates?

@perfortnight
Copy link

No. I've been away.
But on that note, the docker image I have (openworm_test) does not appear to have the file structure that the https://github.com/openworm/OpenWorm/blob/development/master_openworm.py file expects. For example, there is no /home/openworm directory.

Is this the right docker image to be working with? Are there necessary installations to make before it is ready to work with? If so, are they documented somewhere?

@slarson
Copy link
Member Author

slarson commented Jan 31, 2016

@perfortnight ok try again; you are right; the image didn't seem to have that stuff. I have pushed a new version; this time it should work

@mwatts15 mwatts15 pinned this issue May 29, 2019
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

3 participants