jupyter_client icon indicating copy to clipboard operation
jupyter_client copied to clipboard

is_complete_reply status conflicts with general reply status

Open jasongrout opened this issue 5 years ago • 3 comments

In https://jupyter-client.readthedocs.io/en/latest/messaging.html#request-reply, we see that the status in a reply message content can be 'ok', 'error', or 'abort'. However, for is_complete_reply, it lists the status possibilities as one of 'complete', 'incomplete', 'invalid', 'unknown'.

These are in conflict with each other. It would be great to get some clarification.

And perhaps the completion status should be a different content attribute in the next spec version?

jasongrout avatar May 29 '19 14:05 jasongrout

Inspecting a few completion messages from ipykernel, I only see the status field being set to 'ok'. I see no field with a value from the other list (it might be overridden?). Maybe the field for code completion should be renamed?

vidartf avatar May 30 '19 11:05 vidartf

This is the is_complete_reply message, not a completion message. These messages are used, for example, in the console when it is checking to see if the current input is valid and can be run, or if we should wait for more input. For example, make sure the jlab console is set so that it runs on pressing enter. Then type some input and you should see an is_complete_request message.

jasongrout avatar Jun 07 '19 05:06 jasongrout

Ah, sorry for the noise.

vidartf avatar Jun 09 '19 22:06 vidartf