updated BUGS.md and minor changes in the code

Miguel Barão
1 parent 21d1f9e9
Showing 3 changed files with 8 additions and 8 deletions Show diff stats
BUGS.md
app.py
test.py
  
 # BUGS
  
-- numeric interval deve converter respostas que usam virgulas para pontos decimais
 - a revisao do teste não mostra as imagens.
-- se aluno tem teste activo e é allowed uma segunda vez, deve manter o mesmo teste. adicionar opcao para eliminar um teste em curso.
 - melhorar o botao de autorizar (desliga-se), usar antes um botao?
 e.g. retornar None quando nao ha alteracoes relativamente à última vez.
 ou usar push (websockets?)
@@ -16,8 +14,8 @@ ou usar push (websockets?)
  
 # TODO
  
-- decorador para user 0, evita o "if uid==0" em muitas funcoes.
-- gerar teste qd o prof autoriza? melhor nao, pode apagar o teste em curso.
+- adicionar opcao para eliminar um teste em curso.
+- gerar teste qd o prof autoriza? melhor nao, pode apagar o teste em curso. gerar previamente e manter uma pool de testes gerados?
 - enviar resposta de cada pergunta individualmente.
 - experimentar gerador de svg que inclua no markdown da pergunta e ver se funciona. 
 - suportar cotacao to teste diferente de 20 (e.g. para juntar perguntas em papel). opcao "points: 18" que normaliza total para 18 em vez de 20.
@@ -61,6 +59,8 @@ ou usar push (websockets?)
  
 # FIXED
  
+- decorador para user 0, evita o "if uid==0" em muitas funcoes.
+- numeric interval deve converter respostas que usam virgulas para pontos decimais
 - self.testapp.get_json_filename_of_test(test_id) retorna None quando test_id nao existe.
 - o eventloop está a bloquear. correção do teste é blocking. usar threadpoolexecutor?
 - substituir get_event_loop por get_runnint_loop (ver https://docs.python.org/3/library/asyncio-eventloop.html)
@@ -138,10 +138,10 @@ class App(object):
     # -----------------------------------------------------------------------
     async def generate_test(self, uid):
         if uid in self.online:
-            logger.info(f'Student {uid}: generating new test.')
-            student_id = self.online[uid]['student']
+            logger.info(f'Student {uid}: started generating new test.')
+            student_id = self.online[uid]['student']  # {number, name}
             self.online[uid]['test'] = await self.testfactory.generate(student_id)
-            logger.debug(f'Student {uid}: test ok.')
+            logger.debug(f'Student {uid}: finished generating test.')
             return self.online[uid]['test']
         else:
             # this implies an error in the code. should never be here!
@@ -216,7 +216,7 @@ class Test(dict):
         self['finish_time'] = None
         self['state'] = 'ONGOING'
         self['comment'] = ''
-        logger.info(f'Student {self["student"]["number"]}: starting test.')
+        logger.info(f'Student {self["student"]["number"]}:  new test.')
  
     # -----------------------------------------------------------------------
     # Removes all answers from the test (clean)
1	1
2	2	# BUGS
3	3
4		-- numeric interval deve converter respostas que usam virgulas para pontos decimais
5	4	- a revisao do teste não mostra as imagens.
6		-- se aluno tem teste activo e é allowed uma segunda vez, deve manter o mesmo teste. adicionar opcao para eliminar um teste em curso.
7	5	- melhorar o botao de autorizar (desliga-se), usar antes um botao?
8	6	e.g. retornar None quando nao ha alteracoes relativamente à última vez.
9	7	ou usar push (websockets?)
...	...	@@ -16,8 +14,8 @@ ou usar push (websockets?)
16	14
17	15	# TODO
18	16
19		-- decorador para user 0, evita o "if uid==0" em muitas funcoes.
20		-- gerar teste qd o prof autoriza? melhor nao, pode apagar o teste em curso.
	17	+- adicionar opcao para eliminar um teste em curso.
	18	+- gerar teste qd o prof autoriza? melhor nao, pode apagar o teste em curso. gerar previamente e manter uma pool de testes gerados?
21	19	- enviar resposta de cada pergunta individualmente.
22	20	- experimentar gerador de svg que inclua no markdown da pergunta e ver se funciona.
23	21	- suportar cotacao to teste diferente de 20 (e.g. para juntar perguntas em papel). opcao "points: 18" que normaliza total para 18 em vez de 20.
...	...	@@ -61,6 +59,8 @@ ou usar push (websockets?)
61	59
62	60	# FIXED
63	61
	62	+- decorador para user 0, evita o "if uid==0" em muitas funcoes.
	63	+- numeric interval deve converter respostas que usam virgulas para pontos decimais
64	64	- self.testapp.get_json_filename_of_test(test_id) retorna None quando test_id nao existe.
65	65	- o eventloop está a bloquear. correção do teste é blocking. usar threadpoolexecutor?
66	66	- substituir get_event_loop por get_runnint_loop (ver https://docs.python.org/3/library/asyncio-eventloop.html)
...	...
...	...	@@ -138,10 +138,10 @@ class App(object):
138	138	# -----------------------------------------------------------------------
139	139	async def generate_test(self, uid):
140	140	if uid in self.online:
141		- logger.info(f'Student {uid}: generating new test.')
142		- student_id = self.online[uid]['student']
	141	+ logger.info(f'Student {uid}: started generating new test.')
	142	+ student_id = self.online[uid]['student'] # {number, name}
143	143	self.online[uid]['test'] = await self.testfactory.generate(student_id)
144		- logger.debug(f'Student {uid}: test ok.')
	144	+ logger.debug(f'Student {uid}: finished generating test.')
145	145	return self.online[uid]['test']
146	146	else:
147	147	# this implies an error in the code. should never be here!
...	...
...	...	@@ -216,7 +216,7 @@ class Test(dict):
216	216	self['finish_time'] = None
217	217	self['state'] = 'ONGOING'
218	218	self['comment'] = ''
219		- logger.info(f'Student {self["student"]["number"]}: starting test.')
	219	+ logger.info(f'Student {self["student"]["number"]}: new test.')
220	220
221	221	# -----------------------------------------------------------------------
222	222	# Removes all answers from the test (clean)
...	...