Skip to content
This repository was archived by the owner on Feb 4, 2022. It is now read-only.

Update to new Beakerx#72

Merged
hadim merged 2 commits intomasterfrom
update-beakerx
Jun 27, 2017
Merged

Update to new Beakerx#72
hadim merged 2 commits intomasterfrom
update-beakerx

Conversation

@hadim
Copy link
Contributor

@hadim hadim commented Jun 22, 2017

Trying to update to twosigma/beakerx@45ed2ca

Some breaking API changes were trivial. But after that, the kernel doesn't work:

[INFO] Log level used is : info
[INFO] Scijava Kernel is started and ready to use.
[Thread-3] INFO com.twosigma.beakerx.kernel.msg.MessageCreator - Execution result ERROR: The kernel was not ready, run the cell again.
[Thread-3] INFO com.twosigma.beakerx.kernel.msg.MessageCreator - Execution result ERROR: The kernel was not ready, run the cell again.
[E 18:10:06.531 LabApp] Notebook JSON is invalid: Additional properties are not allowed ('text' was unexpected)
    
    Failed validating 'additionalProperties' in error:
    
    On instance['cells'][1]['outputs'][0]:
    {'ename': 'The kernel was not ready, run the cell again.',
     'evalue': 'The kernel was not ready, run the cell again.',
     'output_type': 'error',
     'text': 'The kernel was not ready, run the cell again.',
     'traceback': ['\x1b[0;31mThe kernel was not ready, run the cell again.']}

@scottdraves
Copy link

Hi this is something we are about to take out. I could give a long explanation of what it's about but it doesn't really matter, it was a mistake and we are in the final steps of removing it: twosigma/beakerx#5434

should be early next week. if you want to just work around it, make this one line change to this constructor:

  public EvaluatorManager(KernelFunctionality kernel, Evaluator evaluator) {
    this.kernel = kernel;
    this.evaluator = evaluator;
    evaluator.startWorker();
    this.codeExecutor = this::execute;
  }

@hadim
Copy link
Contributor Author

hadim commented Jun 22, 2017

The update is not urgent. I just wanted to keep up with the base kernel. I can wait until next week.

@hadim
Copy link
Contributor Author

hadim commented Jun 27, 2017

Working now.

@hadim hadim merged commit f843762 into master Jun 27, 2017
@hadim hadim deleted the update-beakerx branch June 27, 2017 14:43
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants