VuisterLab / cing

Automated Validation of NMR Structures
http://nmr.le.ac.uk
2 stars 4 forks source link

CING iPython interactive shell integration broken for some combination #329

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Startup cing with --ipython

What is the expected output? What do you see instead?
jd:duvel/~/ cing --noProject --ipython
================================================================================
======================
| CING: Common Interface for NMR structure Generation version 0.95 (r1147)      
AW,JFD,GWV 2004-2011 |
================================================================================
======================
User: jd         on: duvel.cmbi.umcn.nl (darwin/64bit/4cores/2.6.7) at:  (1939) 
Thu Mar 22 15:03:12 2012
CING started at : Thu Mar 22 15:03:07 2012
CING stopped at : Thu Mar 22 15:03:12 2012
CING took       : 4.551 s

Traceback (most recent call last):
  File "/Users/jd/workspace35/cing/python/cing/main.py", line 1065, in <module>
    main()
  File "/Users/jd/workspace35/cing/python/cing/main.py", line 1043, in main
    from IPython.Shell import IPShellEmbed # optional module; not required for CING proper.
ImportError: No module named Shell

jd:duvel/~/ ipython
Python 2.6.7 (r267:88850, Mar  2 2012, 16:46:38) 
Type "copyright", "credits" or "license" for more information.

IPython 0.12 -- An enhanced Interactive Python.
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.

This is likely an installation error but could become important for future 
compatibility.

Original issue reported on code.google.com by jurge...@gmail.com on 22 Mar 2012 at 2:09

GoogleCodeExporter commented 9 years ago
Thanks to (Alan I think) from:
https://github.com/ipython/ipython/issues/286

and example at:
http://wiki.ipython.org/Cookbook/Updating_code_that_uses_IPython_for_0.11 and
http://wiki.ipython.org/Cookbook/Dynamic_Prompt

I found the work around.

Original comment by jurge...@gmail.com on 23 Mar 2012 at 1:56

GoogleCodeExporter commented 9 years ago

Original comment by jurge...@gmail.com on 26 Mar 2012 at 11:53

GoogleCodeExporter commented 9 years ago
Attempted a fix in r1149 but needs testing in order to see if all machines in 
the install base can handle the new method. If not, I'll have to special case 
it.

Original comment by jurge...@gmail.com on 26 Mar 2012 at 1:24

GoogleCodeExporter commented 9 years ago
Nop, failed because the new system is only installed in IPython 0.12 and not 
0.10. I will need to special case it's use.

Original comment by jurge...@gmail.com on 26 Mar 2012 at 2:00

GoogleCodeExporter commented 9 years ago
Special cased ipython versions 0.10 and 0.12 in r1150.

Now testing.

Original comment by jurge...@gmail.com on 27 Mar 2012 at 10:07

GoogleCodeExporter commented 9 years ago
Updated install on nmrpro to include now mandatory ipython package.
After that all installs tested fine in Jenkins.

Original comment by jurge...@gmail.com on 28 Mar 2012 at 10:10