BhallaLab / moose

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

Connect BioModels/All Curated ModelIds fails: Exception: (307, u'Temporary Redirect') #230

Open yurivict opened 6 years ago

yurivict commented 6 years ago

The progress indicator goes up to 13% and then throws this exception:

Traceback (most recent call last):
  File "/usr/local/lib/moose/gui/biomodelsclient.py", line 240, in runQuery
    name = unicode(self.client.service.getModelNameById(item)).encode("utf-8")
  File "/usr/local/lib/moose/gui/suds/client.py", line 521, in __call__
    return client.invoke(args, kwargs)
  File "/usr/local/lib/moose/gui/suds/client.py", line 581, in invoke
    result = self.send(soapenv)
  File "/usr/local/lib/moose/gui/suds/client.py", line 619, in send
    description=tostr(e), original_soapenv=original_soapenv)
  File "/usr/local/lib/moose/gui/suds/client.py", line 677, in process_reply
    raise Exception((status, description))
Exception: (307, u'Temporary Redirect')
dilawar commented 6 years ago

AFAIK, suds is not being maintained anymore. MOOSE uses suds to download the model. @hrani (and probably @subhacom ) might be able to comment on if there is plan to continue with suds to drop it altogether in favor of 'curl' or urllib2 etc.

subhacom commented 6 years ago

curl or urllib2 is not the same thing as suds, which is a SOAP client. There is a fork of suds which has been maintained irregularly, and if anybody is willing to port the code, zeep might be a better alternative at present. However I wonder how many users are out there who might want this feature.

dilawar commented 6 years ago

Well, IMO either a feature should be maintained or removed. Let @hrani and @upibhalla take a call on this one. zeep looks promising; it depends only on python-lxml which is a well-maintained library and available on almost all linux flavors for which we build pacakges.

hrani commented 6 years ago

Yes I need to see zeep across the platform, if again not supported across the platform not worth migrating. Will discuss with Upi and take a call.

On Fri, 29 Dec 2017 at 9:06 AM, Dilawar Singh notifications@github.com wrote:

Well, IMO either a feature should be maintained or removed. Let @hrani https://github.com/hrani and @upibhalla https://github.com/upibhalla take a call on this one. zeep looks promising; it depends only on python-lxml which is a well-maintained library and available on almost all linux flavors for which we build pacakges.

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/BhallaLab/moose/issues/230#issuecomment-354393418, or mute the thread https://github.com/notifications/unsubscribe-auth/ALFhw7Fk__kXSP1bejHCDsUmjoVKTN5Sks5tFF5KgaJpZM4RGFsn .

dilawar commented 6 years ago

@hrani Is it planned for chennapoda?

hrani commented 6 years ago

Will check and let you know.

On Tue, 6 Mar 2018 at 4:52 PM, Dilawar Singh notifications@github.com wrote:

@hrani https://github.com/hrani Is it planned for chennapoda?

— You are receiving this because you were mentioned.

Reply to this email directly, view it on GitHub https://github.com/BhallaLab/moose/issues/230#issuecomment-370750079, or mute the thread https://github.com/notifications/unsubscribe-auth/ALFhwy7-puBIAddhdz1TGkxQIDc8tTPVks5tbnFjgaJpZM4RGFsn .

dilawar commented 6 years ago

@hrani Is they any update on this ticket?

dilawar commented 4 years ago

Pinging if there is update here.