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 'Nonetype' not iterable for ObsidianLoader #12726

Closed
wants to merge 0 commits into from

Conversation

Di3mex
Copy link

@Di3mex Di3mex commented Nov 1, 2023

When the frontmatter of an Obsidian note contains a tags key which has an empty value an error is raised as a 'Nonetype' is being iterated. This leads to the Loader failing.

---
date: 2023-10-17 13:59
modification_date: 2023-10-17 13:59
tags: 
---

Copy link

vercel bot commented Nov 1, 2023

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

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
langchain ⬜️ Ignored (Inspect) Visit Preview Nov 1, 2023 6:52pm

@dosubot dosubot bot added Ɑ: doc loader Related to document loader module (not documentation) 🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature labels Nov 1, 2023
@Di3mex Di3mex changed the title Fixes 'Nonetype' not iterable Fixes 'Nonetype' not iterable for ObsidianLoader Nov 1, 2023
@efriis efriis self-assigned this Nov 1, 2023
@efriis efriis closed this Nov 1, 2023
@efriis
Copy link
Member

efriis commented Nov 1, 2023

Hey @Di3mex - running into some issues with your fork. Will do this in a new PR!

efriis added a commit that referenced this pull request Nov 1, 2023
xieqihui pushed a commit to xieqihui/langchain that referenced this pull request Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖:bug Related to a bug, vulnerability, unexpected error with an existing feature Ɑ: doc loader Related to document loader module (not documentation)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants