annotate jupyter_c_kernel/kernel.py @ 35:e05900c7967e

Remove logging
author Brendan Rius <brendan@omixy.com>
date Sat, 30 Apr 2016 22:47:49 +0100
parents b5bc8483ac3e
children 8acbab5a9f21
Ignore whitespace changes - Everywhere: Within whitespace: At end of lines:
rev   line source
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
1 from queue import Queue, Empty
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
2 from threading import Thread
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
3
0
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
4 from ipykernel.kernelbase import Kernel
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
5 import subprocess
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
6 import tempfile
2
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
7 import os
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
8
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
9
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
10 class JupyterSubprocess(subprocess.Popen):
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
11 def __init__(self, cmd, write_to_stdout, write_to_stderr):
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
12 self._write_to_stdout = write_to_stdout
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
13 self._write_to_stderr = write_to_stderr
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
14
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
15 super().__init__(cmd, stdout=subprocess.PIPE, stderr=subprocess.PIPE)
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
16
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
17 self._stdout_queue = Queue()
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
18 self._stdout_thread = Thread(target=JupyterSubprocess._enqueue_output, args=(self.stdout, self._stdout_queue))
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
19 self._stdout_thread.daemon = True
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
20 self._stdout_thread.start()
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
21
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
22 self._stderr_queue = Queue()
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
23 self._stderr_thread = Thread(target=JupyterSubprocess._enqueue_output, args=(self.stderr, self._stderr_queue))
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
24 self._stderr_thread.daemon = True
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
25 self._stderr_thread.start()
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
26
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
27 @staticmethod
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
28 def _enqueue_output(contents, queue):
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
29 for line in iter(contents.readline, b''):
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
30 queue.put(line)
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
31 contents.close()
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
32
35
e05900c7967e Remove logging
Brendan Rius <brendan@omixy.com>
parents: 34
diff changeset
33 def write_contents(self):
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
34 try:
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
35 stdout_contents = self._stdout_queue.get_nowait()
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
36 except Empty:
35
e05900c7967e Remove logging
Brendan Rius <brendan@omixy.com>
parents: 34
diff changeset
37 pass
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
38 else:
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
39 self._write_to_stdout(stdout_contents)
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
40 try:
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
41 stderr_contents = self._stderr_queue.get_nowait()
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
42 except Empty:
35
e05900c7967e Remove logging
Brendan Rius <brendan@omixy.com>
parents: 34
diff changeset
43 pass
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
44 else:
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
45 self._write_to_stderr(stderr_contents)
0
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
46
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
47
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
48 class CKernel(Kernel):
20
696cc0775abb Change package name
Brendan Rius <brendan@omixy.com>
parents: 9
diff changeset
49 implementation = 'jupyter_c_kernel'
0
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
50 implementation_version = '1.0'
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
51 language = 'c'
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
52 language_version = 'C11'
2
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
53 language_info = {'name': 'c',
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
54 'mimetype': 'text/plain',
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
55 'file_extension': 'c'}
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
56 banner = "C kernel.\n" \
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
57 "Uses gcc, compiles in C11, and creates source code files and executables in temporary folder.\n"
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
58
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
59 def __init__(self, *args, **kwargs):
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
60 super(CKernel, self).__init__(*args, **kwargs)
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
61 self.files = []
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
62
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
63 def cleanup_files(self):
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
64 """Remove all the temporary files created by the kernel"""
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
65 for file in self.files:
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
66 os.remove(file)
0
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
67
2
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
68 def new_temp_file(self, **kwargs):
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
69 """Create a new temp file to be deleted when the kernel shuts down"""
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
70 # We don't want the file to be deleted when closed, but only when the kernel stops
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
71 kwargs['delete'] = False
9
101e99452042 Dockerized project :)
Brendan Rius <brendan@omixy.com>
parents: 7
diff changeset
72 kwargs['mode'] = 'w'
2
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
73 file = tempfile.NamedTemporaryFile(**kwargs)
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
74 self.files.append(file.name)
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
75 return file
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
76
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
77 def _write_to_stdout(self, contents):
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
78 self.send_response(self.iopub_socket, 'stream', {'name': 'stdout', 'text': contents})
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
79
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
80 def _write_to_stderr(self, contents):
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
81 self.send_response(self.iopub_socket, 'stream', {'name': 'stderr', 'text': contents})
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
82
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
83 def create_jupyter_subprocess(self, cmd):
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
84 return JupyterSubprocess(cmd,
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
85 lambda contents: self._write_to_stdout(contents.decode()),
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
86 lambda contents: self._write_to_stderr(contents.decode()))
2
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
87
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
88 def compile_with_gcc(self, source_filename, binary_filename):
2
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
89 args = ['gcc', source_filename, '-std=c11', '-o', binary_filename]
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
90 return self.create_jupyter_subprocess(args)
0
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
91
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
92 def do_execute(self, code, silent, store_history=True,
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
93 user_expressions=None, allow_stdin=False):
2
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
94 with self.new_temp_file(suffix='.c') as source_file:
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
95 source_file.write(code)
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
96 source_file.flush()
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
97 with self.new_temp_file(suffix='.out') as binary_file:
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
98 p = self.compile_with_gcc(source_file.name, binary_file.name)
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
99 while p.poll() is None:
35
e05900c7967e Remove logging
Brendan Rius <brendan@omixy.com>
parents: 34
diff changeset
100 p.write_contents()
e05900c7967e Remove logging
Brendan Rius <brendan@omixy.com>
parents: 34
diff changeset
101 p.write_contents()
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
102 if p.returncode != 0: # Compilation failed
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
103 self._write_to_stderr(
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
104 "[C kernel] GCC exited with code {}, the executable will not be executed".format(
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
105 p.returncode))
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
106 return {'status': 'ok', 'execution_count': self.execution_count, 'payload': [],
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
107 'user_expressions': {}}
0
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
108
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
109 p = self.create_jupyter_subprocess([binary_file.name])
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
110 while p.poll() is None:
35
e05900c7967e Remove logging
Brendan Rius <brendan@omixy.com>
parents: 34
diff changeset
111 p.write_contents()
e05900c7967e Remove logging
Brendan Rius <brendan@omixy.com>
parents: 34
diff changeset
112 p.write_contents()
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
113
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
114 if p.returncode != 0:
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
115 self._write_to_stderr("[C kernel] Executable exited with code {}".format(p.returncode))
2
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
116 return {'status': 'ok', 'execution_count': self.execution_count, 'payload': [], 'user_expressions': {}}
7
aa54c85303b6 Remove temporary files when shutting down the kernel
Brendan Rius <brendan@omixy.com>
parents: 4
diff changeset
117
aa54c85303b6 Remove temporary files when shutting down the kernel
Brendan Rius <brendan@omixy.com>
parents: 4
diff changeset
118 def do_shutdown(self, restart):
aa54c85303b6 Remove temporary files when shutting down the kernel
Brendan Rius <brendan@omixy.com>
parents: 4
diff changeset
119 """Cleanup the created source code files and executables when shutting down the kernel"""
aa54c85303b6 Remove temporary files when shutting down the kernel
Brendan Rius <brendan@omixy.com>
parents: 4
diff changeset
120 self.cleanup_files()