Skip to content
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

Fixes #1207 #1208

Merged
merged 2 commits into from
Dec 6, 2024
Merged

Fixes #1207 #1208

merged 2 commits into from
Dec 6, 2024

Conversation

kbarendrecht
Copy link
Contributor

@kbarendrecht kbarendrecht commented Dec 4, 2024

I encountered a ModuleNotFoundError: No module named 'nest_asyncio' error after installing the deepeval library. It seems that nest_asyncio, which is required by the library, is not automatically installed as a dependency.

see #1207

either this or a JetBrains.gitignore should be added
Copy link

vercel bot commented Dec 4, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
evals-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 6, 2024 10:50am

@kbarendrecht
Copy link
Contributor Author

@kritinv @penguine-ip would you mind taking a look? it's a tiny change.

@penguine-ip
Copy link
Contributor

@kbarendrecht hey thanks for the bump I missed it - one quick question what's the ./idea folder in gitignore?

@kbarendrecht
Copy link
Contributor Author

Hi @penguine-ip, that's the configuration folder for Jetbrains IDE's. it creates it on opening a folder automaticly. I didn't want to force my settings onto anyone. It's recommened to exclude the entire thing or parts of it.

@see https://github.com/github/gitignore/blob/main/Global/JetBrains.gitignore#L4

@penguine-ip
Copy link
Contributor

@kbarendrecht oh no not at all was just curious about the extra directory, thanks for the PR

@penguine-ip penguine-ip merged commit d7a85a7 into confident-ai:main Dec 6, 2024
3 of 4 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