Skip to content

cannot import name 'InsertIntoLLM' from 'llm_strategy' #4

Description

@keunwoochoi

hi, i just tried to use

Traceback (most recent call last):
  File "/Users/choik11/.venv/bin/markitdown", line 4, in <module>
    from openize.markitdown.main import main
  File "/Users/choik11/.venv/lib/python3.11/site-packages/openize/markitdown/__init__.py", line 8, in <module>
    from .processor import DocumentProcessor
  File "/Users/choik11/.venv/lib/python3.11/site-packages/openize/markitdown/processor.py", line 4, in <module>
    from llm_strategy import SaveLocally, InsertIntoLLM
ImportError: cannot import name 'InsertIntoLLM' from 'llm_strategy' (/Users/choik11/.venv/lib/python3.11/site-packages/openize/markitdown/llm_strategy.py)

just confirmed in the source that InsertIntoLLM doesn't exist in the file.

i guess in general, the test coverage can be improved to include cases like this.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

No labels
No labels

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions