Opened 10 years ago

Closed 5 years ago

#9785 closed defect (wontfix)

Simple Server API - wrong cell results

Reported by: dpoetzsch Owned by: jason, was
Priority: major Milestone: sage-duplicate/invalid/wontfix
Component: notebook Keywords:
Cc: jason Merged in:
Authors: Reviewers: Karl-Dieter Crisman
Report Upstream: N/A Work issues:
Branch: Commit:
Dependencies: Stopgaps:

Description

Using the Simple Server API I tried the following (after logging in of course):

At first I sent the code "sleep(10);5" to the server using the following URL: http://localhost:port/simple/compute?session=theID&code=sleep(10)%3B5

Then, immediatly afterwards (this means command one was still computing) I sent the code "4+5" to the server using the URL http://localhost:port/simple/compute?session=theID&code=4%2B5

Then, after 10 seconds, I looked at the results of both commands using http://localhost:port/simple/status?session=theID&cell=2 and http://localhost:port/simple/status?session=theID&cell=3

Both cells had 5 as result, which should be only the result of the first cell.

Attachments (1)

trac_9785_fix_wrong_cell.patch (995 bytes) - added by mmanashirov 8 years ago.
This fixes the problem for me.

Download all attachments as: .zip

Change History (6)

Changed 8 years ago by mmanashirov

This fixes the problem for me.

comment:1 Changed 7 years ago by kcrisman

  • Milestone set to sage-duplicate/invalid/wontfix
  • Reviewers set to Karl-Dieter Crisman
  • Status changed from new to needs_review

Sadly, the simple server does not work with the current notebook, and the Sage cell server has more or less superseded it.

comment:2 Changed 7 years ago by kcrisman

  • Cc jason added
  • Status changed from needs_review to positive_review

Although we'll close this, I hope the code here will someday prove useful, though!

comment:3 Changed 7 years ago by kcrisman

  • Status changed from positive_review to needs_info

Note that #11409 would remove this completely.

comment:4 Changed 5 years ago by kcrisman

  • Status changed from needs_info to positive_review

This has now been removed.

comment:5 Changed 5 years ago by vbraun

  • Resolution set to wontfix
  • Status changed from positive_review to closed
Note: See TracTickets for help on using tickets.