HazyResearch / evaporate

This repo contains data and code for the paper "Language Models Enable Simple Systems for Generating Structured Views of Heterogeneous Data Lakes"
475 stars 45 forks source link

follow the readme cannot work #2

Closed fenglixa closed 1 year ago

fenglixa commented 1 year ago

The steps in the readme file are not clear, and some errors with following the steps:

After run bash run.sh Report error:

Cannot import name 'Manifest' from 'manifest'
simran-arora commented 1 year ago

Hi thanks for your note. Did you install manifest already?

fenglixa commented 1 year ago

Yes, I installed it, followed the steps as below:

git clone git@github.com:HazyResearch/evaporate.git
cd evaporate
# pip install -r requirements.txt

# Weak supervision code
cd metal-evap
git submodule init
git submodule update
pip install -e .

# Manifest 
git clone git@github.com:HazyResearch/manifest.git
cd manifest
pip install -e .

git lfs install
git clone https://huggingface.co/datasets/hazyresearch/evaporate

bash run.sh

And report error as below:

bash run.sh
Traceback (most recent call last):
  File "run_profiler.py", line 11, in <module>
    from utils import get_structure, get_manifest_sessions, get_file_attribute
  File "/root/test/evaporate/utils.py", line 5, in <module>
    from manifest import Manifest
ImportError: cannot import name 'Manifest' from 'manifest' (unknown location)
Traceback (most recent call last):
  File "run_profiler.py", line 11, in <module>
    from utils import get_structure, get_manifest_sessions, get_file_attribute
  File "/root/test/evaporate/utils.py", line 5, in <module>
    from manifest import Manifest
ImportError: cannot import name 'Manifest' from 'manifest' (unknown location)
simran-arora commented 1 year ago

Did the pull request fix your issue? It seems like manifest is running for you now?

fenglixa commented 1 year ago

Yeah, let's close this issue now. Thanks! /close

fenglixa commented 1 year ago

Open again to trace the readme file still need to be updated. e.g. the requirement.txt file is not exist. Maybe later, I could have some inputs.

simran-arora commented 1 year ago

Ah good catch, I've updated the README. Let me know if you still run into issues and thanks for your help

fenglixa commented 1 year ago

Thanks @simran-arora /Close this issue

raywgs commented 1 year ago

hey how did you fix the Cannot import name 'Manifest' from 'manifest' issue?