BhallaLab / moose

MOOSE simulator.
http://moose.ncbs.res.in
GNU General Public License v3.0
25 stars 19 forks source link

Rdesigneur lxml import #198

Closed iampritishpatil closed 8 years ago

iampritishpatil commented 8 years ago

In line 30 and 31 uses lxml. This modules need not be installed and is not in the requirements.

#########################################################################
## rdesigneur0_5.py ---
## This program is part of 'MOOSE', the
## Messaging Object Oriented Simulation Environment.
##           Copyright (C) 2014 Upinder S. Bhalla. and NCBS
## It is made available under the terms of the
## GNU General Public License version 2 or later.
## See the file COPYING.LIB for the full notice.
#########################################################################

##########################################################################
## This class builds models of
## Reaction-Diffusion and Electrical SIGnaling in NEURons.
## It loads in neuronal and chemical signaling models and embeds the
## latter in the former, including mapping entities like calcium and
## channel conductances, between them.
##########################################################################
from __future__ import print_function
import imp
import os
import moose
import numpy as np
import pylab
import math
import rmoogli
#import rdesigneurProtos
from rdesigneurProtos import *
from moose.neuroml.NeuroML import NeuroML
from moose.neuroml.ChannelML import ChannelML
import lxml
from lxml import etree
import h5py as h5
import csv

It should either be added to dependencies or there should be:

from moose.neuroml.ChannelML import ChannelML
try:
    from lxml import etree
except:
    from xml import etree
import h5py as h5
dilawar commented 8 years ago

This has been fixed by https://github.com/BhallaLab/moose-core/commit/52ae361cf5f2f7c6ab76c62b4ea903ca895a639b

Also see BhallaLab/moose-core#147