Hints and plans for permute/2 added.

2 files changed, 21 insertions, 43 deletions

diff --git a/prolog/problems/lists/permute_2/common.py b/prolog/problems/lists/permute_2/common.py
index 3035159..55c2b48 100644
--- a/prolog/problems/lists/permute_2/common.py
+++ b/prolog/problems/lists/permute_2/common.py
@@ -30,7 +30,6 @@ hint_type = {
     'recursive_case': Hint('recursive_case'),
     'timeout': Hint('timeout'),
     'base_case_arbitrary': Hint('base_case_arbitrary'),
-    'second_base_case_missing': Hint('second_base_case_missing'),
     'unsuccessful_conc_use': Hint('unsuccessful_conc_use'),
     'forcing_result_onto_recursion': Hint('forcing_result_onto_recursion'),
 }
@@ -85,26 +84,17 @@ def hint(code, aux_code):
 
 
         # forcing result onto recursion; case of divide(T, [H1|L1], [H2|L2])
-        if not prolog.engine.ask_truthTO(engine_id, 'divide([a,b,c], _, _)') and \
+        if not prolog.engine.ask_truthTO(engine_id, 'permute([a,b,c], _)') and \
            prolog.engine.ask_truthTO(engine_id,
-           'asserta( divide([cob], [yowza,cob], [brix]) ), divide([yowza,brix,cob], [cob], []), retract( divide([cob], [yowza,cob], [brix]) )'):
+           'asserta( permute([cob], [yowza,cob]) ), permute([yowza,cob], [cob]), retract( permute([cob], [yowza,cob]) )'):
             return [{'id': 'forcing_result_onto_recursion'}]
 
         # base case succeeds with arbitrary result
-        if prolog.engine.ask_truthTO(engine_id, 'divide([], L1, L2), (var(L1) ; var(L2))') or \
-           prolog.engine.ask_truthTO(engine_id, 'divide([q], L1, L2), (var(L1) ; var(L2))') or \
-           prolog.engine.ask_truthTO(engine_id, 'divide([q], [q|L1], L2), (var(L1) ; var(L2))'):
+        if prolog.engine.ask_truthTO(engine_id, 'permute([], P), var(P)'):
             return [{'id': 'base_case_arbitrary'}]
 
-        # one base case ok, the other one is missing (or bad)
-        # only trigger when recursion also works, because that makes more sense to the student
-        # than immediately telling him or her that there is another base case missing
-        if prolog.engine.ask_truthTO(engine_id,
-           'divide([a,b], [a], [b]), \+ divide([a,b,c], [a,c], [b]) ; \+ divide([a,b], [a], [b]), divide([a,b,c], [a,c], [b])'):
-            return [{'id': 'second_base_case_missing'}]
-
         # missing/failed base case
-        if not prolog.engine.ask_truth(engine_id, 'divide([], [], [])'):
+        if not prolog.engine.ask_truth(engine_id, 'permute([], [])'):
             return [{'id': 'base_case'}]
 
 #        # target predicate seems to always be false
@@ -114,7 +104,7 @@ def hint(code, aux_code):
 
         # base cases work, the recursive doesn't (but it doesn't timeout)
         # this may be left as the last, most generic hint
-        if not prolog.engine.ask_truth(engine_id, 'divide([qa,qb,qc,qd], [qa,qc], [qb,qd])'):
+        if not prolog.engine.ask_truth(engine_id, 'permute([qa,qb], [qb,qa])'):
             return [{'id': 'recursive_case'}]
 
     except socket.timeout as ex:
diff --git a/prolog/problems/lists/permute_2/sl.py b/prolog/problems/lists/permute_2/sl.py
index e565262..c712f5d 100644
--- a/prolog/problems/lists/permute_2/sl.py
+++ b/prolog/problems/lists/permute_2/sl.py
@@ -15,22 +15,18 @@ description = '''\
   L = [3,2,1].
 </pre>'''
 
-#<p><img src="[%@resource plan.svg%]" /></p>
-#''', '''\
-
 
 plan = ['''\
 <p><img src="[%@resource plan.svg%]" /></p>
-<p>Saj veš kako je šlo v osnovni šoli: prvi, drugi, prvi, ...</p>
-''', '''\
-<p>Znaš vzeti dva elementa z začetka seznama? Vzorec je <code>[H1,H2|T]</code>.</p>
+<p>Poskusi rekurzirvno pripraviti eno možnost, a pusti prologu svobodo. Le-ta bo potem sam od sebe
+vračal vse možne rešitve.</p>
 ''', '''\
-<p>Vzameš dva elementa z začetka, preostanek rekurzivno razdeliš in to, kar vrne rekurzija, primerno
-dopolniš s prej odvzetima elementoma. S tem, ko si vzel dva elementa z začetka, si problem zmanjšal.</p>
+<p>Možnosti je več, a ena je naslednja: kaj, če vzamem prvi element stran, problem je s tem manjši in
+ga lahko prepustim rekurziji, potem pa ta element vstavim. Kaj ga pa vstavim? Vstavi ga na vse možne načine.</p>
 ''', '''\
-<p>Če predpostavim, da rekurzija razdeli rep <code>T</code> na podseznama <code>L1</code> in <code>L2</code>
-ter ob vračanju v <code>L1</code> na začetek dodam <code>H1</code> in v <code>L2</code> na začetek
-dodam <code>H2</code>, potem sem razdelil začetni seznam, ki je oblike <code>[H1,H2|T]</code>.</p>
+<p>Če predpostavim, da rekurzija permutira rep <code>T</code> in vrne "permutirani" rep <code>PT</code> in
+ob vračanju v <code>PT</code> nekam vstavim glavo <code>H</code>, potem bom dobil neko permutacijo začetnega
+seznama, ki je oblike <code>[H|T]</code>.</p>
 ''']
 
 hint = {
@@ -46,35 +42,27 @@ implicitno že kar v argumentih predikata (glavi stavka).</p>
 ''',
 
     'base_case': '''\
-<p>Si pomislil na robni pogoj? Kaj je najbolj enostaven primer? Kaj, če je seznam prazen?</p>
+<p>Si pomislil na robni pogoj? Kaj je najbolj enostaven primer? Kaj, če je seznam kar prazen?</p>
 ''',
 
     'base_case_arbitrary': '''\
-<p>Kako je lahko rezultat delitve seznama poljuben seznam oz. karkoli?</p>
-<p>Če je tvoj robni pogoj v stilu <code>divide([], _, _)</code> ali <code>divide([X], [X|_], ...)</code>,
-ga še enkrat premisli: kaj je rezultat delitve, kaj vračaš? Robni pogoj je vedno dokončno specificirana
+<p>Kako je lahko rezultat permutiranja seznama poljuben seznam oz. karkoli?</p>
+<p>Če je tvoj robni pogoj podoben <code>permute([], _)</code>,
+ga še enkrat premisli: kaj je rezultat permutiranja, kaj vračaš? Robni pogoj je vedno dokončno specificirana
 rešitev, tu načeloma ni neznank (<code>_</code> ali spremenljivk brez prirejenih vrednosti) v tem kar se vrača.</p>
 ''',
 
-    'second_base_case_missing': '''\
-<p>Rekurzija se ne konča vedno uspešno. Sta morda dva različna primera kako se lahko izteče? Saj veš,
-sodo in liho ;) Je morda potreben še kakšen robni pogoj? Poskusi naslednja klica, eden bo uspešen, drugi pa ne:</p>
-<p><code>?- divide([a,b,c], L1, L2).</code></p>
-<p><code>?- divide([a,b,c,d], L1, L2).</code></p>
-''',
-
     'unsuccessful_conc_use': '''\
-<p>Uporabljaš <code>conc/3</code>? Pri tej nalogi to ni najboljša ideja, ker <code>conc/3</code> deli "v kosih",
-težko boš prišel do posameznih elementov. Poskusi raje brez.</p>
+<p>Uporabljaš <code>conc/3</code>? Pri tej nalogi to ni najboljša ideja, morda pa pride prav kakšna druga že
+rešena naloga.</p>
 ''',
 
     'forcing_result_onto_recursion': '''
 <p>Ne vsiljuj rekurziji kaj naj vrne, prepusti se ji. To je tisti del, ko narediš predpostavko,
 če je ta izpolnjena, potem bo tvoje pravilo delovalo za večji primer.</p>
-<p>Je tvoj rekurzivni klic oblike <code>divide(T, [H1|...], [H2|...])</code>? S tem vsiljuješ rekurziji
-da mora <emph>vrniti</emph> tudi obe glavi, ki jih sploh ne pozna, ker si jih ravnokar vzel stran! To moraš
-narediti ti z rezultati, ki jih rekurzija vrne. Skratka, elementa <code>H1</code> in <code>H2</code>
-dodaj izven rekurzivnega klica.</p>
+<p>Je tvoj rekurzivni klic oblike <code>permute(T, [H|...])</code>? S tem vsiljuješ rekurziji
+da mora <emph>vrniti</emph> tudi glavo, ki je sploh ne pozna, ker si jo ravnokar vzel stran! To moraš
+narediti ti z rezultatom, ki ga rekurzija vrne. Skratka, element <code>H</code> dodaj izven rekurzivnega klica.</p>
 ''',
 
     'recursive_case': '''\