-
Notifications
You must be signed in to change notification settings - Fork 31
Refactor DPP script #168
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
base: main
Are you sure you want to change the base?
Refactor DPP script #168
Conversation
Signed-off-by: Rafael Vasquez <rafvasq21@gmail.com>
45f661c to
8e99ab7
Compare
Signed-off-by: Rafael Vasquez <rafvasq21@gmail.com>
Signed-off-by: Rafael Vasquez <rafvasq21@gmail.com>
Signed-off-by: Rafael Vasquez <rafvasq21@gmail.com>
Signed-off-by: Rafael Vasquez <rafvasq21@gmail.com>
4642142 to
340c383
Compare
Signed-off-by: Rafael Vasquez <rafvasq21@gmail.com>
Signed-off-by: Rafael Vasquez <rafvasq21@gmail.com>
Signed-off-by: Rafael Vasquez <rafvasq21@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Worth putting a main function too so that we can do something like this?
if __name__ == "__main__":
main()
That might simplify this too: https://github.com/foundation-model-stack/aiu-fms-testing-utils/blob/main/pyproject.toml#L49
Signed-off-by: Rafael Vasquez <rafvasq21@gmail.com>
|
Also worth checking if the above changes affect the project.scripts: https://github.com/foundation-model-stack/aiu-fms-testing-utils/blob/main/pyproject.toml#L49 |
eed279a to
0146eb7
Compare
Signed-off-by: Rafael Vasquez <rafvasq21@gmail.com>
3abee83 to
3debfea
Compare
Refactors the DPP script primarily to improve code structure and readability.
valid_promptsand separate types of tests (metric,token, etc.)