Skip to content

add write utils#1601

Merged
penguine-ip merged 4 commits into
confident-ai:mainfrom
daehuikim:writeconfig
May 20, 2025
Merged

add write utils#1601
penguine-ip merged 4 commits into
confident-ai:mainfrom
daehuikim:writeconfig

Conversation

@daehuikim

Copy link
Copy Markdown
Contributor

When I use deepeval with massive data without web application, It is difficult to track every results from the printed logs.
To address this, I hope to add file writing functions on the utils to get log for evaluation.
This function even helps to track missing data by connection errors.

@vercel

vercel Bot commented May 15, 2025

Copy link
Copy Markdown

@daehuikim is attempting to deploy a commit to the Confident AI Team on Vercel.

A member of the Team first needs to authorize it.

@penguine-ip

Copy link
Copy Markdown
Contributor

hey! @daehuikim can we put this in display config? thanks!

@daehuikim

daehuikim commented May 16, 2025

Copy link
Copy Markdown
Contributor Author

@penguine-ip Yes please. :)
I noticed my commit is missing adding variable in display_config.
I will make extra commit about this. <= Done

@penguine-ip

Copy link
Copy Markdown
Contributor

Hey @daehuikim thanks for changing it, i was hoping to have everything in utils. An optional file_output_dir would be great for the display config, what do you think?

@daehuikim

Copy link
Copy Markdown
Contributor Author

@penguine-ip Can you confirm the change from the last commit please? I hope I am on the right way.
I just made file_output_dir in the DisplayConfig and everything is happen in the utils.py

@penguine-ip

Copy link
Copy Markdown
Contributor

@daehuikim this is perfect, thanks!

@penguine-ip penguine-ip merged commit 4e82e8a into confident-ai:main May 20, 2025
1 of 3 checks passed
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