python-holidays icon indicating copy to clipboard operation
python-holidays copied to clipboard

Please add holidays from Prophet

Open ourownstory opened this issue 4 years ago • 4 comments

Thank you for this great library. We at NeuralProphet would like to be able to rely 100% on your great package. However, you are lacking a few important countries, which we have to manually sync from Prophet. If you could add those, it would make NeuralProphet's and Prophet's codebase simpler to maintain.

Prophet supports the following countries missing in your library:

  • Indonesia
  • Thailand
  • Philippines
  • Pakistan
  • Belarus
  • India (currently defined: 8 holidays, while Prophet has 22)

You can find the relevant code here: https://github.com/facebook/prophet/blob/master/python/prophet/hdays.py

"All that is needed" is to write tests for these countries, the holidays are all already defined.

If someone would be down to take on this task, we would be very thankful!

ourownstory avatar Oct 07 '21 18:10 ourownstory

Hi @ourownstory thank you for your interest in python_holidays.

It would be great if you could contribute to it, if you can, thx!

In the meanwhile, I mark this as "help wanted".

Cheers!

dr-prodigy avatar Nov 13 '21 11:11 dr-prodigy

Hi @dr-prodigy I am happy to contribute everything but the tests...

ourownstory avatar Nov 15 '21 02:11 ourownstory

I can work on it

tserekh avatar Nov 17 '21 13:11 tserekh

Thank you @tserekh ! Please let me know when your work is merged!

ourownstory avatar Mar 03 '22 19:03 ourownstory

The requested countries have been added in v0.20. Please review https://github.com/ourownstory/neural_prophet/pull/1176

Thank you for using python-holidays!

arkid15r avatar Feb 28 '23 00:02 arkid15r