Skip to content

Keeping README.md consistent with code #9

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Conversation

nishika26
Copy link

This pull request includes updates to the README with key changes to simplify the workflow:

  • Replaced Environment Variable with Direct File Path: The earlier approach, where the evaluations.jsonl file path was passed via an environment variable ($evaluations.jsonl), is now replaced with a direct file path (/path/to/evaluations.jsonl) in the command. If we would continue using it as an environment variable, it would require an additional step to first export the evaluation JSON file path to an environment variable before being able to use the previous command.

  • Updated Script Paths: The paths for the plotting scripts have been updated to reflect changes in the folder structure, moving from src/analysis/plot-scores.py to src/analysis/end-to-end/plot-scores.py and src/analysis/plot-responses.py to src/analysis/response-scores/plot-scores.py.

@nishika26 nishika26 requested a review from AkhileshNegi June 26, 2025 18:16
@nishika26 nishika26 self-assigned this Jun 26, 2025
@nishika26 nishika26 added the documentation Improvements or additions to documentation label Jun 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant