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

Add WND-CHARM Docker image #27

Closed
wants to merge 2 commits into from
Closed

Conversation

manics
Copy link
Member

@manics manics commented Jan 26, 2016

Part of the work on testing WND-CHARM in the IDR. Don't merge yet.

@colettace Can you remind me how to run the tests?

@colettace
Copy link

@manics you mean this?

cd tests/pywndcharm_tests
python -m unittest discover

@manics
Copy link
Member Author

manics commented Jan 26, 2016

Does this look familiar?

[wndcharm@b7ddf323bff3 pywndcharm_tests]$ python test_FeatureVector.py
Traceback (most recent call last):
  File "test_FeatureVector.py", line 221, in <module>
    from wndcharm.FeatureSpace import FeatureSpace
  File "/usr/lib64/python2.7/site-packages/wndcharm-0.9.11-py2.7-linux-x86_64.egg/wndcharm/FeatureSpace.py", line 55, in <module>
    class FeatureSpace( object ):
  File "/usr/lib64/python2.7/site-packages/wndcharm-0.9.11-py2.7-linux-x86_64.egg/wndcharm/FeatureSpace.py", line 68, in FeatureSpace
    channel_col_finder = re.compile(r'(?P<path>.*?)?\{(?P<opts>.*?)?\}')
  File "/usr/lib64/python2.7/re.py", line 190, in compile
    return _compile(pattern, flags)
  File "/usr/lib64/python2.7/re.py", line 242, in _compile
    raise error, v # invalid expression
sre_constants.error: nothing to repeat

@colettace
Copy link

what does cat /etc/redhat-release and python --version say?

@manics
Copy link
Member Author

manics commented Jan 26, 2016

[wndcharm@b7ddf323bff3 ~]$ cat /etc/redhat-release
CentOS Linux release 7.2.1511 (Core)
[wndcharm@b7ddf323bff3 ~]$ python --version
Python 2.7.5

Installation steps are all in https://github.com/ome/ome-docker/pull/27/files#diff-6f1358368f9abe505c8ef01514b8e886

@colettace
Copy link

That error does look familiar:

http://stackoverflow.com/questions/5869650/python-regex-strange-behavior

We got it on Linux when using Python 2.6, so we stopped supporting. I thought it was a 2.6 vs. 2.7 problem but now I see earlier versions of Python 2.7 are also susceptible.

Well, upgrading Python to the latest version should fix it, but maybe I can look into changing it so it will be backwards-compatible?

Although it has the same version of Python (2.7.5) as Centos:7 it appears to work with WND-CHARM, maybe it has a back-ported patch?
@manics
Copy link
Member Author

manics commented Jan 28, 2016

I thought I'd try Ubuntu 14.04. It has the same Python version (2.7.5) but appears to more or less work. A few tests are failing, but doesn't look too bad.

@joshmoore
Copy link
Member

cc: @simleo in case you're interested.

@simleo
Copy link
Member

simleo commented Apr 27, 2016

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

Successfully merging this pull request may close these issues.

4 participants