Skip to content

reponses file inconsistent variable name#4

Merged
seebi merged 9 commits intomainfrom
evaluatebug
May 2, 2025
Merged

reponses file inconsistent variable name#4
seebi merged 9 commits intomainfrom
evaluatebug

Conversation

@paulorvdc
Copy link
Member

tried to run it today before looking into creating tests tomorrow and found a simple mistake with a variable named differently from the click command.
also fixed another problem I created because I changed how I wanted to declare the output file to be consistent with the ask command and forgot part of the old code.

@github-actions
Copy link

github-actions bot commented May 1, 2025

Coverage

Coverage Report
File Stmts Miss Cover Missing
init.py 0 0 100%
cli.py 16 0 100%
context.py 7 0 100%
database.py 33 2 94% 76-77
request.py 20 3 85% 52-53 56
commands/init.py 0 0 100%
commands/ask.py 48 5 90% 94-98
commands/evaluate.py 61 4 93% 37 105-107
commands/serve.py 29 8 72% 41 47-50 82-84
models/init.py 0 0 100%
models/questions_file.py 24 0 100%
models/response.py 6 0 100%
utils/init.py 0 0 100%
utils/evaluation_metrics.py 33 3 91% 32-34
utils/query_rdf.py 13 3 77% 28-30
TOTAL 290 28 90%  

Tests Skipped Failures Errors Time
14 0 💤 0 ❌ 0 🔥 12.957 ⏱

@paulorvdc paulorvdc requested review from seebi and removed request for seebi May 2, 2025 11:56
@seebi seebi merged commit c5676cd into main May 2, 2025
2 checks passed
@seebi seebi deleted the evaluatebug branch May 2, 2025 16:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants