Closed ndrmahmoudi closed 7 years ago
Thanks for mentioning this! I made the default DATA path relative so that you don't need to play with the constants to load the lexicons (assuming you are running commands from the root directory of the repo).
Of course, you still need to download the embeddings and set those paths manually.
I still get the same error. I am sure that I running from root. When I install the package using pip, it does not create any folder for lexicons or representations. There, I get the error.
Ahh I see. The pip installation was indeed messed up. The pip installation should work for the lexicon data now, as I have made that data a part of the package distribution. However, you will still need to modify constants.py for the embedding paths (and if you install via pip, then you need to make this modification wherever pip installed the source). The README now recommends the python setup.py install
route rather than pip install due to this mess.
Ideally, the user should be able to pip install and then run a command to download the default word vectors, but this is a todo enhancement at this point.
I was trying to modify the constant.py
for the embedding paths and realised that following folders are missed from the data folder:
LEXICONS = DATA + 'lexicon_info/'
PROCESSED_LEXICONS = DATA + 'lexicons/'
POLARITIES = DATA + 'polarities/'
The data folder in github contains just the lexicons.
Whoops, this minor follow-up slipped under my radar. I removed the LEXICONS folder because it contains info that is not useful to downstream users. The POLARITIES folder is just a default output folder (e.g., for the code in the historical dir) that should be automatically created. There is no actual data in it by default. I modified the constants.py so that the code automatically checks if that directory exists and creates it if necessary on an import.
Hi again William,
I appreciate your quick responses. Regarding the lexicons, I got following error:
I know that I have to play with paths in
constant.py
file. I just wanted to let you know about the bug.Regards, Nader