opiniondigest icon indicating copy to clipboard operation
opiniondigest copied to clipboard

which is the python version required for this code?

Open VijayIyer opened this issue 3 years ago • 4 comments

  1. When I do 'pip install -r requirements.txt', I get errors when pandas version starts to download, and pip install of dependencies stops.
  2. Is there equivalent code to this with latest versions of torchtext and torch?

VijayIyer avatar Nov 01 '21 02:11 VijayIyer

Hi @VijayIyer : Facing similar kind of issues, have you found answers for your questions. If so, kindly share Thanks in advance.

HarithaMK14 avatar Dec 28 '21 15:12 HarithaMK14

Same problem here, pandas 0.22.0 is only compatible with python 2.7, 3.5 and 3.6, while torch 1.2.0 doesn't work with those versions for me

EricvanSchaik avatar Feb 28 '22 13:02 EricvanSchaik

I manually installed the version specific packages. There was actually quite a few workarounds I had to do. This was a while back so I don't recall very well. Below are some changes I had to make to make it work.

importing gensim for gensim.downloader as api installed gensim==3.8.3 aggregate.py -> api.load for w2v takes time add encoding=utf-8 for each read

..............error in generate.py from sumeval.metrics.rouge import RougeCalculator from sumeval.metrics.bleu import BLEUCalculator ...........

...installed sumeval in generate.py

....sacrebleu 1.5.1

....needed to make prepare_default to prepare_yelp-default midway

...get_agg_name in utils.py....change in name reading

.......torchtext/utils.py line 130 replacing maxInt in place of sys.maxsize...for windows??

....csv.writer takes newline="" argument

VijayIyer avatar Feb 28 '22 15:02 VijayIyer

To get the packages to work, I used python 3.6.4, sumeval 0.1.7 and also sacrebleu 1.5.1. and gensim 3.8.3 To get the right PyTorch package I also had to provide the download link, so pip install torch==1.2.0 -f https://download.pytorch.org/whl/torch_stable.html (I use Windows and PyPi didn't have the torch 1.2.0 windows binaries). I also used torchtext 0.5.0 to get rid of the maxInt overflow bug.

I also managed to fix most of the other bugs (altough it still doesn't fully work on the default dataset). I pushed it to my fork https://github.com/EricvanSchaik/opiniondigest

EricvanSchaik avatar Mar 02 '22 10:03 EricvanSchaik