UW-xDD / text2graph_llm

An experimental API endpoint to convert text to knowledge graph triplets.
MIT License
2 stars 1 forks source link

Improve docs and binary handling #37

Closed JasonLo closed 2 months ago

JasonLo commented 2 months ago

Summary of Changes:

@ilmcconnell I've made some changes to geolocation/macrostrat.py, please review. The main reason for this adjustment is that I encountered a write permission issue on HTCondor, where users cannot write directly to the installed Python package directory. This was disrupting the download process, so I've now included the macrostrat data as a binary file.

JasonLo commented 2 months ago

@iross please review at readme

ilmcconnell commented 2 months ago

@JasonLo added in tests for geocoding, removed old llm tests, got all tests passing, and separated dev dependencies

ilmcconnell commented 2 months ago

@JasonLo local binary for stratname records a good solution. Only caveat is assuming macrostrat doesn't change much ever - @iross true?