annotate jupyter_c_kernel/kernel.py @ 36:8acbab5a9f21

Remove stream buffering on stdout & stderr
author Louis 'Kureuil' Person <louis.person@epitech.eu>
date Sat, 30 Apr 2016 23:48:19 +0200
parents e05900c7967e
children aac47bc07111
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
36
8acbab5a9f21 Remove stream buffering on stdout & stderr
Louis 'Kureuil' Person <louis.person@epitech.eu>
parents: 35
diff changeset
8 import os.path as path
34
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 = []
36
8acbab5a9f21 Remove stream buffering on stdout & stderr
Louis 'Kureuil' Person <louis.person@epitech.eu>
parents: 35
diff changeset
62 mastertemp = tempfile.mkstemp(suffix='.out')
8acbab5a9f21 Remove stream buffering on stdout & stderr
Louis 'Kureuil' Person <louis.person@epitech.eu>
parents: 35
diff changeset
63 os.close(mastertemp[0])
8acbab5a9f21 Remove stream buffering on stdout & stderr
Louis 'Kureuil' Person <louis.person@epitech.eu>
parents: 35
diff changeset
64 self.master_path = mastertemp[1]
8acbab5a9f21 Remove stream buffering on stdout & stderr
Louis 'Kureuil' Person <louis.person@epitech.eu>
parents: 35
diff changeset
65 filepath = path.join(path.dirname(path.realpath(__file__)), '..', 'resources', 'master.c')
8acbab5a9f21 Remove stream buffering on stdout & stderr
Louis 'Kureuil' Person <louis.person@epitech.eu>
parents: 35
diff changeset
66 subprocess.call(['gcc', filepath, '-std=c11', '-fPIC', '-shared', '-rdynamic', '-o', self.master_path])
2
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
67
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
68 def cleanup_files(self):
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
69 """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
70 for file in self.files:
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
71 os.remove(file)
36
8acbab5a9f21 Remove stream buffering on stdout & stderr
Louis 'Kureuil' Person <louis.person@epitech.eu>
parents: 35
diff changeset
72 os.remove(self.master_path)
0
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
73
2
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
74 def new_temp_file(self, **kwargs):
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
75 """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
76 # 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
77 kwargs['delete'] = False
9
101e99452042 Dockerized project :)
Brendan Rius <brendan@omixy.com>
parents: 7
diff changeset
78 kwargs['mode'] = 'w'
2
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
79 file = tempfile.NamedTemporaryFile(**kwargs)
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
80 self.files.append(file.name)
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
81 return file
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
82
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
83 def _write_to_stdout(self, contents):
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
84 self.send_response(self.iopub_socket, 'stream', {'name': 'stdout', 'text': contents})
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
85
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
86 def _write_to_stderr(self, contents):
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
87 self.send_response(self.iopub_socket, 'stream', {'name': 'stderr', 'text': contents})
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
88
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
89 def create_jupyter_subprocess(self, cmd):
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
90 return JupyterSubprocess(cmd,
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
91 lambda contents: self._write_to_stdout(contents.decode()),
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
92 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
93
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
94 def compile_with_gcc(self, source_filename, binary_filename):
36
8acbab5a9f21 Remove stream buffering on stdout & stderr
Louis 'Kureuil' Person <louis.person@epitech.eu>
parents: 35
diff changeset
95 args = ['gcc', source_filename, '-std=c11', '-fPIC', '-shared', '-rdynamic', '-o', binary_filename]
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
96 return self.create_jupyter_subprocess(args)
0
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
97
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
98 def do_execute(self, code, silent, store_history=True,
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
99 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
100 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
101 source_file.write(code)
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
102 source_file.flush()
b46b2e5b6c08 Improve error management by separating stdout and stderr
Brendan Rius <brendan@omixy.com>
parents: 0
diff changeset
103 with self.new_temp_file(suffix='.out') as binary_file:
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
104 p = self.compile_with_gcc(source_file.name, binary_file.name)
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
105 while p.poll() is None:
35
e05900c7967e Remove logging
Brendan Rius <brendan@omixy.com>
parents: 34
diff changeset
106 p.write_contents()
e05900c7967e Remove logging
Brendan Rius <brendan@omixy.com>
parents: 34
diff changeset
107 p.write_contents()
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
108 if p.returncode != 0: # Compilation failed
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
109 self._write_to_stderr(
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
110 "[C kernel] GCC exited with code {}, the executable will not be executed".format(
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
111 p.returncode))
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
112 return {'status': 'ok', 'execution_count': self.execution_count, 'payload': [],
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
113 'user_expressions': {}}
0
50ea00cf5896 Initial commit
Brendan Rius <brendan@omixy.com>
parents:
diff changeset
114
36
8acbab5a9f21 Remove stream buffering on stdout & stderr
Louis 'Kureuil' Person <louis.person@epitech.eu>
parents: 35
diff changeset
115 p = self.create_jupyter_subprocess([self.master_path, binary_file.name])
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
116 while p.poll() is None:
35
e05900c7967e Remove logging
Brendan Rius <brendan@omixy.com>
parents: 34
diff changeset
117 p.write_contents()
e05900c7967e Remove logging
Brendan Rius <brendan@omixy.com>
parents: 34
diff changeset
118 p.write_contents()
33
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
119
Brendan Rius <brendan@omixy.com>
parents: 29
diff changeset
120 if p.returncode != 0:
34
Brendan Rius <brendan@omixy.com>
parents: 33
diff changeset
121 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
122 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
123
aa54c85303b6 Remove temporary files when shutting down the kernel
Brendan Rius <brendan@omixy.com>
parents: 4
diff changeset
124 def do_shutdown(self, restart):
aa54c85303b6 Remove temporary files when shutting down the kernel
Brendan Rius <brendan@omixy.com>
parents: 4
diff changeset
125 """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
126 self.cleanup_files()