datasciencecampus / pygrams

Extracts key terminology (n-grams) from any large collection of documents (>1000) and forecasts emergence
https://datasciencecampus.github.io/pygrams
Other
62 stars 23 forks source link

pygrams requires date of (timeseries_)date_to, date_from in "yyyy/mm/dd" format but this breaks file writing on windows. #385

Open l-bailey-fnc opened 1 year ago

l-bailey-fnc commented 1 year ago

Describe the bug Pygrams requires the data_to, date_from, timeseries_date_to, and timeseries_date_from command line arguments to be passed provided in YYYY/MM/DD format, but this breas the output file creation (on windows at least).

Change to use YYYY-MM-DD format. (Note, this is the superior date format!).

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Click on '....'
  3. Scroll down to '....'
  4. See error

Expected behavior A clear and concise description of what you expected to happen.

Screenshots If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

Smartphone (please complete the following information):

Additional context Add any other context about the problem here.