From 898789199e6af91dfa900650c22df6d26f7e635f Mon Sep 17 00:00:00 2001
From: Martin <martin@leo.fri1.uni-lj.si>
Date: Thu, 24 Sep 2015 12:39:32 +0200
Subject: Finished the first version of introduction section.

---
 python/problems/introduction/ballistics/common.py |  61 +++++--------
 python/problems/introduction/ballistics/sl.py     | 106 +++++++++-------------
 2 files changed, 67 insertions(+), 100 deletions(-)

(limited to 'python/problems/introduction/ballistics')

diff --git a/python/problems/introduction/ballistics/common.py b/python/problems/introduction/ballistics/common.py
index f7f271e..10ef692 100644
--- a/python/problems/introduction/ballistics/common.py
+++ b/python/problems/introduction/ballistics/common.py
@@ -1,11 +1,12 @@
 # coding=utf-8
 
-from python.util import has_token_sequence, string_almost_equal
+from python.util import has_token_sequence, string_almost_equal, \
+    string_contains_number, get_tokens, get_numbers, get_exception_desc
 from server.hints import Hint, HintSequence
 
 id = 187
 group = 'introduction'
-number = 2
+number = 3
 visible = True
 
 solution = '''\
@@ -21,34 +22,19 @@ print("Kroglo bo odneslo", razdalja, "metrov.")
 hint_type = {
     'plan': Hint('plan'),
     'eval_expression': Hint('eval_expression'),
-    'sin_error': HintSequence('sin_error', 4),
-    'name_error': HintSequence('name_error', 4),
-    'unsupported_operand': HintSequence('unsupported_operand', 4),
-    'error': HintSequence('error', 2),
-    'radians': HintSequence('radians', 3),
+    'sin_error': Hint('sin_error'),
+    'name_error': Hint('name_error'),
+    'unsupported_operand': Hint('unsupported_operand'),
+    'error': Hint('error'),
+    'radians': Hint('radians'),
     'printing': Hint('printing'),
     'betterg': Hint('betterg'),
 }
 
-import re
-numeric_const_pattern = r"""
-    [-+]? # optional sign
-    (?:
-         (?: \d* \. \d+ ) # .1 .12 .123 etc 9.1 etc 98.1 etc
-         |
-         (?: \d+ \.? ) # 1. 12. 123. etc 1 12 123 etc
-    )
-    # followed by optional exponent part if desired
-    (?: [Ee] [+-]? \d+ ) ?
-    """
-rx = re.compile(numeric_const_pattern, re.VERBOSE)
-
 def contains_negative(s):
-    """ Returns whether the string contains negative value or not.
-    """
-    vals = rx.findall(s)
-    for v in vals:
-        if float(v) < 0:
+    nums = get_numbers(s)
+    for n in nums:
+        if n < 0:
             return True
     return False
 
@@ -89,20 +75,29 @@ def test(python, code):
     outputs = [ans[1] for ans in answers]
 
     n_correct = 0
-    for output, correct in zip(outputs, test_out):
+    tin = None
+    for i, (output, correct) in enumerate(zip(outputs, test_out)):
+        print (output, float(correct))
         if string_almost_equal(output, float(correct)):
             n_correct += 1
+        else:
+            tin = test_in[i][1].replace('\n', ' ')
+            tout = correct
 
     passed = n_correct == len(test_in)
     hints = [{'id': 'test_results', 'args': {'passed': n_correct, 'total': len(test_in)}}]
+    if tin:
+        hints.append({'id': 'problematic_test_case', 'args': {'testin': str(tin), 'testout': str(tout)}})
     return passed, hints
 
 def hint(python, code):
+    tokens = get_tokens(code)
+
     # run one test first to see if there are any exceptions
     test_in = [(None, '5\n10\n')]
     answer = python(code=code, inputs=test_in, timeout=1.0)
     exc = answer[0][3]
-    # if have an exception!
+    exc_hint = get_exception_desc(answer[0][3])
     if exc:
         if 'sin' in exc and 'NameError' in exc:
             return [{'id':'sin_error', 'args': {'message': exc}}]
@@ -113,21 +108,15 @@ def hint(python, code):
         elif 'TypeError' in exc:
             return [{'id':'type_error', 'args': {'message': exc}}]
         else:
-            return [{'id':'error', 'args': {'message': exc}}]
-
-    # show plan if student is lost
-    # a) empty progam
-    # b) there is not input (we can do it here, since we have no input hint)
-    if not code or (not has_token_sequence(code, ['input'])):
-        return [{'id': 'plan'}]
+            return exc_hint
 
     # if sinus is not in code, we need to teach students where they can get it
     # use math functions.
-    if  (not has_token_sequence(code, ['sin'])):
+    if  (not has_token_sequence(tokens, ['sin'])):
         return [{'id' : 'eval_expression'}]
 
     # student is not using print function
-    if not has_token_sequence(code, ['print']):
+    if not has_token_sequence(tokens, ['print']):
         return [{'id' : 'printing'}]
 
     # if result is negative, student did not translate to radians
diff --git a/python/problems/introduction/ballistics/sl.py b/python/problems/introduction/ballistics/sl.py
index 15b02be..c271ecb 100644
--- a/python/problems/introduction/ballistics/sl.py
+++ b/python/problems/introduction/ballistics/sl.py
@@ -1,4 +1,6 @@
 # coding=utf-8
+import server
+mod = server.problems.load_language('python', 'sl')
 
 id = 187
 name = 'Topologija'
@@ -7,76 +9,54 @@ slug = 'Topologija'
 description = '''\
 <p>Napiši program za izračun dolžine strela s topom (ki brez trenja izstreljuje 
 točkaste krogle v brezzračnem prostoru, a pustimo trivio). Program od uporabnika 
-ali uporabnice zahteva, da vnese hitrost izstrelka (to je, omenjene točkaste krogle) 
-in kot, pod katerim je izstreljen. Program naj izračuna in izpiše, kako daleč 
+ali uporabnice zahteva, da vnese kot, pod katerim je izstreljen, in hitrost izstrelka. Program naj izračuna in izpiše, kako daleč
 bo letela krogla.</p>
 <p>Pomoč za fizično nebogljene: s=v<sup>2</sup>sin(2f)/g , kjer je s razdalja, 
-v hitrost izstrelka, f je kot, g pa osma črka slovenske abecede.</p>
+v hitrost izstrelka, f je kot, g pa gravitacijska konstanta (9,8).</p>
 <p>Preveri tole: krogla leti najdalj, če jo izstrelimo pod kotom 45 stopinj. 
 Poskusi, kako daleč gre pod kotom 45 in kako daleč pod 46 stopinj -- po 45 mora leteti dlje. 
 Preveri tudi kot 50 stopinj: če pod tem kotom leti nazaj (razdalja je negativna), 
 si ga gotovo nekje polomil.
+<p><i>Opomba: za pravilno delovanje moraš v programu najprej prebrati kot in potem hitrost.</i></p>
 '''
 
-general_exception = {
-    'error_head' : '''\
-<p>Napaka:</p>
-<pre>
-[%=message%]
-</pre>
-''',
-
-    'general': '''
-    <p>Pri razumevanju napake sta pomembni dve vrstici. V predzadnji vrstici je napisana lokacija
-    napake (line ...), v zadnji vrstici pa izvemo za kakšno napako gre.</p>
-    ''',
-
-    'name_error' : '''
-    <p>Napaka <code>NameError</code> pomeni, da uporabljate nedefinirano vrednost:
-    ali vrednost spremenljivke ni določena ali uporabljate funkcijo, ki ni uvožena.</p>
-    ''',
-
-    'type_error': '''
-    <p>TypeError napaka pomeni, da želite izvesti operacijo na nedovoljenih tipih.
-    Npr., če želite sešteti niz in število ali klicati funkcijo, čeprav tisto ni funkcija, itd. .</p>
-    ''',
-
-    'indentation_error': '''
-    <p> Pomeni, da imate napačno število presledkov na začetku vrstice. Poskrbite, da so vse vrstice pravilno
-poravnane. </p>'''
-
-}
-
-hint = {
-    'plan': '''\
-<p>Plan sledi že znani strategiji: preberi vrednosti – izračunaj – izpiši.</p>
-''',
-    
-    'eval_expression': '''
-<p>Izračunati morate pot po formuli: s=v<sup>2</sup>sin(2f)/g</p>
-<p>Konstanta g je gravitacijski pospešek in ima vrednost približno 9.8 :) </p>
-<p>Funkcijo <code>sin</code> dobite v modulu <code>math</code>, tako kot smo tam 
+eval_expression = ['''\
+<p>Dolžino strela dobiš po formuli: s=v<sup>2</sup>sin(2f)/g.</p>''',
+                   '''\
+<p>Funkcijo <code>sin</code> dobiš v modulu <code>math</code>, tako kot smo tam
 dobili funkcij <code>sqrt</code>. Na začetku programa  torej napišite:</p>
 <pre>
 from math import *
-</pre>
-<p>Če vam funkcija <code>sin</code> ne deluje pravilno, vam svetujemo, da si 
-pogledate v dokumentacijo, kako funkcija deluje (še posebno bodite pozorni na 
+</pre>''',
+                   '''\
+<p>Če vam funkcija <code>sin</code> ne deluje pravilno, vam svetujemo, da si
+pogledate v dokumentacijo, kako funkcija deluje (še posebno bodite pozorni na
 razlago v oklepajih). Do dokumentacije dostopate z ukazom help:</p>
 <pre>
 help(sin)
 </pre>
-''',
+'''
+]
 
-    'printing': '''
-<p> V Pythonu izpisujemo s funkcijo <code>print. Če želimo izpisati več elementov, 
-jih ločimo z vejico.  Recimo, da imamo spremenljivko <code>ime</code>, 
-ki vsebuje naše ime, potem lahko napišemo:
-<pre>
-print("Ime mi je", ime, ".")
-</pre>''',
+printing = ['''\
+<p> V Pythonu izpisujemo s funkcijo <code>print</code>.</p>''']
 
-    'radians': ['''
+plan = ['''\
+<p>Plan sledi že znani strategiji: preberi vrednosti – izračunaj – izpiši.</p>''',
+        eval_expression,
+        printing
+
+]
+
+hint = {
+
+    'eval_expression': eval_expression,
+
+    'printing': printing,
+
+    'radians': ['''\
+<p>Poskusi <code>sin(30)</code>. Zakaj je rezultat negativen?</p>''',
+                '''\
 <p>Vse triginometrične funkcije sprejemajo kot v radianih in ne v stopinjah.</p>''', 
 ''' <p>V stopinjah ima cel krog 360°, v radianih pa 2*pi. Zato je 360° ekvivalentno 2*pi.</p>''',
 ''' <p>Formula za pretvorbo med stopinjami in radiani je: </p>
@@ -84,34 +64,32 @@ kot_rad = kot * 2 * pi / 360'''],
 
     'betterg': '''Konstanta g naj ima vrednost 9.8''',
 
-    'name_error' : [general_exception['error_head'], general_exception['general'],
-    general_exception['name_error'], '''
+    'name_error' : [mod.general_msg['error_head'], mod.general_msg['general_exception'],
+    mod.general_msg['name_error'], '''
     <p>Verjetno uporabljate spremenljivko, ki nima vrednosti. Ali v izrazu za izračun
      uporabljate napačno spremenljivko? Ali pri izpisu morda poskušate
      izpisati napačno spremenljivko?</p>'''],
 
-    'sin_error' : [general_exception['error_head'], general_exception['general'],
-    general_exception['name_error'], '''
+    'sin_error' : [mod.general_msg['error_head'], mod.general_msg['general_exception'],
+    mod.general_msg['name_error'], '''
     <p>Za uporabo funkcije <code>sin</code> je potrebno prej uvoziti modul <code>math</code>:
     <code>from math import *</code> ali <code>import math</code>. Če uporabljate slednji način,
     morate vedno uporabiti ime modula pri klicu funkcije: <code>math.sin(...)</code>.'''],
 
-    'unsupported_operand' : [general_exception['error_head'], general_exception['general'],
-    general_exception['type_error'], '''
+    'unsupported_operand' : [mod.general_msg['error_head'], mod.general_msg['general_exception'],
+    mod.general_msg['type_error'], '''
 <p>Verjetni razlog: funkcija <code>input</code> vrača vrednost tipa niz,
 ki jo moramo najprej pretvoriti v tip <code>float</code>, če želimo z njo računati:</p>
 <pre>
 v = float(input(" ...
 </pre>
 '''],
-    'type_error' : [general_exception['error_head'], general_exception['general'],
-    general_exception['type_error'], '''
+    'type_error' : [mod.general_msg['error_head'], mod.general_msg['general_exception'],
+    mod.general_msg['type_error'], '''
 <p>Verjetni razlog: funkcija <code>input</code> vrača vrednost tipa niz,
 ki jo moramo najprej pretvoriti v tip <code>float</code>, če želimo z njo računati:</p>
 <pre>
 v = float(input(" ...
 </pre>
-'''],
-
-    'error' : [general_exception['error_head'], general_exception['general'],]
+''']
 }
-- 
cgit v1.2.1