Skip to content

Conversation

@fxlrnrpt
Copy link
Member

No description provided.

Comment on lines +67 to +70
parser = argparse.ArgumentParser(description="Count accuracy for JSONL files with gold and answer fields")
parser.add_argument("jsonl_path", type=str, help="Path to the JSONL file")

args = parser.parse_args()
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we allow CLI use for information/description files?

Copy link
Member Author

Choose a reason for hiding this comment

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

It is not really an experiment, but a post-processing tool. Should be fine in my view

@@ -0,0 +1,9 @@
import pandas as pd

df = pd.read_json("data/out/distillation/mmlu_synth_gptoss_a_t0_8.jsonl", lines=True)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Switching to read_ndjson will save us a couple of minutes :)

Copy link
Collaborator

Choose a reason for hiding this comment

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

Wouldn't it be better to stick to one file per experiment?
Files for all experiments will also be attached.
Alternative: describe key changes in the experiments in plain text (either in .md or as comments in the code itself).

@fxlrnrpt fxlrnrpt merged commit 0a56823 into main Nov 26, 2025
@fxlrnrpt fxlrnrpt deleted the temperature-test branch November 26, 2025 16:03
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