Skip to content

Evaluation result paper#104

Merged
djarecka merged 50 commits into
performance_cost_optimizationfrom
evaluation_result_paper
Apr 6, 2026
Merged

Evaluation result paper#104
djarecka merged 50 commits into
performance_cost_optimizationfrom
evaluation_result_paper

Conversation

@tekrajchhetri

Copy link
Copy Markdown
Collaborator

This PR contains the following + one minor fix for resource merge.

Benchmark

This directory contains benchmark datasets based on standard evaluation sets.
It includes NCBI and S800, where the .jsonl files serve as the ground-truth annotations.

Link:
https://github.com/sensein/structsense/tree/evaluation_result_paper/evaluation/benchmark


NER

This directory contains evaluation results for three papers related to Named Entity Recognition (NER).

Link:
https://github.com/sensein/structsense/tree/evaluation_result_paper/evaluation/ner


Resource Extraction

This directory contains evaluation results for two papers focused on resource extraction.

Link:
https://github.com/sensein/structsense/tree/evaluation_result_paper/evaluation/resource_extraction


Pdf2Reproschema

This directory contains evaluation outputs for the Pdf2Reproschema pipeline.

Link:
https://github.com/sensein/structsense/tree/evaluation_result_paper/evaluation/pdf2reproschema

@tekrajchhetri
tekrajchhetri requested a review from yibeichan March 26, 2026 20:36
@tekrajchhetri tekrajchhetri mentioned this pull request Mar 26, 2026

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the documentation, configuration files, and evaluation results for the StructSense NER pipeline. The changes include adding an evaluation section to the README, updating configuration files for various models (Gemini, GPT, Qwen), and refining the output structure for evaluation results. The review comments regarding missing trailing newlines in various files are valid and should be addressed to ensure POSIX compliance.

@@ -0,0 +1 @@
{} No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The file is missing a newline character at the end. It's good practice to include a trailing newline for POSIX compliance and better compatibility with various tools.

Suggested change
{}
{}

@@ -0,0 +1,3 @@
{
"judge_method": "auto_approved"
} No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The file is missing a newline character at the end. It's good practice to include a trailing newline for POSIX compliance and better compatibility with various tools.

  "judge_method": "auto_approved"
}

config:
api_base: http://localhost:11434
model: nomic-embed-text

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The file is missing a newline character at the end. It's good practice to include a trailing newline for POSIX compliance and better compatibility with various tools.

    model: nomic-embed-text

config:
api_base: http://localhost:11434
model: nomic-embed-text

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The file is missing a newline character at the end. It's good practice to include a trailing newline for POSIX compliance and better compatibility with various tools.

    model: nomic-embed-text

config:
api_base: http://localhost:11434
model: nomic-embed-text

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The file is missing a newline character at the end. It's good practice to include a trailing newline for POSIX compliance and better compatibility with various tools.

    model: nomic-embed-text

config:
api_base: http://localhost:11434
model: nomic-embed-text

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The file is missing a newline character at the end. It's good practice to include a trailing newline for POSIX compliance and better compatibility with various tools.

    model: nomic-embed-text

config:
api_base: http://localhost:11434
model: nomic-embed-text

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The file is missing a newline character at the end. It's good practice to include a trailing newline for POSIX compliance and better compatibility with various tools.

    model: nomic-embed-text

Comment thread evaluation/ner/readme.md
--config config_name.yaml \
--source input_pdf.pdf \
--api_key sk-<your_api_key>
``` No newline at end of file

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The file is missing a newline character at the end. It's good practice to include a trailing newline for POSIX compliance and better compatibility with various tools.

Suggested change
```
--api_key sk-<your_api_key>

config:
api_base: http://localhost:11434
model: nomic-embed-text:v1.5
model: nomic-embed-text

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The file is missing a newline character at the end. It's good practice to include a trailing newline for POSIX compliance and better compatibility with various tools.

    model: nomic-embed-text

config:
api_base: http://localhost:11434
model: nomic-embed-text:v1.5
model: nomic-embed-text

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The file is missing a newline character at the end. It's good practice to include a trailing newline for POSIX compliance and better compatibility with various tools.

    model: nomic-embed-text

@djarecka
djarecka merged commit 0b1bb04 into performance_cost_optimization Apr 6, 2026
@yibeichan

Copy link
Copy Markdown
Contributor

wait this one is not merged into main... do we not want it to be in main? @tekrajchhetri @djarecka

@tekrajchhetri

tekrajchhetri commented Apr 10, 2026

Copy link
Copy Markdown
Collaborator Author

@yibeichan create a new branch from fix_json_op_issue branch and create a PR. fix_json_op_issue will be merged to improvement and improvement to main. Not yet done.

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.

3 participants