Skip to content

Commit a4613a2

Browse files
Peter JohnsonPeter Johnson
authored andcommitted
Shannon model coding cleanup
1 parent 123a817 commit a4613a2

File tree

3 files changed

+3
-7
lines changed

3 files changed

+3
-7
lines changed

evaluation_function/dev.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
"context_window": 5
2121
},
2222
"shannon_words_ngram": {
23-
"response": "the general",
23+
"response": "the general sweetness",
2424
"answer": 2,
2525
"model": "shannon_words_ngram",
2626
"word_count": 12,

evaluation_function/models/shannon_letters_ngram.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ def run(response, answer, params:Params) -> Result:
9090
word_lengths["tokens"] = [row[0] for row in data]
9191
word_lengths["weights"] = [row[1] for row in data]
9292
word_count = params.get("word_count", 10)
93-
response_used = isinstance(response, int)
93+
response_used = isinstance(response, int) and response > 1
9494
context_window = response if response_used else params.get("context_window", 3)
9595
if word_count == "random":
9696
word_count = random.randint(3,15)
@@ -99,6 +99,6 @@ def run(response, answer, params:Params) -> Result:
9999
output.append(generate_word(k,context_window))
100100
preface = 'Context window: '+str(context_window)+', Word count: '+str(word_count)+'. Output: <br>'
101101
feedback_items = [("general", preface + ' '.join(output))]
102-
feedback_items.append("| Answer not an integer; used default context window") if not response_used else None
102+
feedback_items.append("| Answer not an integer >1; used default context window") if not response_used else None
103103
is_correct = True
104104
return Result(is_correct=is_correct,feedback_items=feedback_items)

evaluation_function/models/shannon_words_ngram.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,6 @@ def get_counts(n=3):
5555
if os.path.exists(FILE):
5656
with open(FILE, "rb") as f:
5757
cache = pickle.load(f)
58-
print(len(cache))
5958
else:
6059
cache = {}
6160
if n not in cache:
@@ -88,9 +87,7 @@ def generate(start="", max_len=20, n=None):
8887
ctx = tuple((([START]*need) + start_tokens)[-need:]) if need else ()
8988
out = start_tokens[:]
9089
for _ in range(max_len):
91-
print(f'next; len(counts): {len(counts)}, ctx: {ctx}')
9290
w = sample_next(counts, ctx)
93-
print(f'w = {w}')
9491
if w in (None, END):
9592
out.append('#')
9693
break
@@ -116,5 +113,4 @@ def run(answer, response, params:Params) -> Result:
116113
feedback_items = [("general", preface + ' '.join(output))]
117114
feedback_items.append("| Answer not an integer; used default context window") if not response_used else None
118115
is_correct = True
119-
print(feedback_items)
120116
return Result(is_correct=is_correct,feedback_items=feedback_items)

0 commit comments

Comments
 (0)