search

Thijsvanede / DeepCASE

Original implementation and resources of DeepCASE as in the S&P '22 paper
MIT License
90 stars 26 forks source link

Just reporting a trivial bug and its solution. Correct me if im wrong in my understanding please. #6

Closed Ammar-Amjad closed 1 year ago

Ammar-Amjad commented 1 year ago

Describe the bug I ran the command from DeepCASE documentation: -> python3 deepcase sequence --txt C:\DeepCASE\data\hdfs_train.txt --save-sequences sequences.save but it threw error Error

Then I just corrected the preprocessor's method from .txt to .text in the main python file and it fixed the error. Correct

I suppose this is the correct working.

To Reproduce Steps to reproduce the behavior:

  1. Just clone the git repo
  2. Follow the commands on the website
  3. The command below will throw error: python3 deepcase sequence --txt --save-sequences sequences.save

image

Desktop (please complete the following information):

Additional context Just highlighting as its a trivial error and can be fixed easily. And if I'm wrong, I would love to be corrected about proper usage of DeepCASE.

Thijsvanede commented 1 year ago

Thank you very much for the bug report, you are indeed correct this should be preprocessor.text(), I will update this in the repository :+1: