Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update of Indian calendar 2024 #1929

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Borgomi42
Copy link
Contributor

compared to Bloomberg INR swap calendar.

@coveralls
Copy link

Coverage Status

coverage: 72.448% (-0.006%) from 72.454%
when pulling 54dbace on Borgomi42:feature/indiaCal2024
into 8b687cb on lballabio:master.

Comment on lines +436 to +437
// Republic Day
|| (d == 26 && m == January)
Copy link
Contributor

@gbfredrik gbfredrik Mar 18, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,
I'm not sure all of these need to be added as they duplicate the base holiday logic at the top:
image

I find it adds complexity here. Without adding them, we are already seeing correct holidays for 2023 and 2024, excluding the Muharrem holiday in July this year, as listed here: https://www.nseindia.com/resources/exchange-communication-holidays. If BBG's INR calendar is different, perhaps we could do with adding another market in the enum within this class? I.e. one to match NSE's holidays (NT calendar in BBG, the current source for this implementation), and one to match RBI's calendar (RB in BBG), as they seem to differ in some years.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would that correspond to "Trading holidays" and "Clearing holidays" at https://www.nseindia.com/resources/exchange-communication-holidays?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let me check...

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That may be the case. I'll gladly help verify when I have access to Bloomberg again after the holidays.

I still hope that we could simplify the logic, dropping those already covered by the standard annual holidays and some others that fall on weekends in 2024:
image
Would it should suffice to only mention them in the comments for 2024?

There is also a recent announcement for an election holiday in May: https://www.livemint.com/market/stock-market-news/lok-sabha-elections-2024-indian-bourses-likely-to-remain-shut-on-polling-day-in-mumbai-11710637030326.html. This can probably be added as well. 🙂

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi again,
I finally got down to reviewing the calendars.

I found one historical occasion where the RBI (Mumbai branch) differed from the NSE (clearing holidays). E.g. 2022-02-07 was a holiday in the former, though not the latter. This caused some turmoil, apparently.

Unless there are objections, I'd be happy to incorporate the improvements from this PR in a new PR, with proper attributions of course. Then I can also get the election holiday (2024-05-20) added. Would that be ok with you?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you. Unfortunately, I have not had time to look in that. Feel free to do so.

Repository owner deleted a comment from FinancialEngineerLab Mar 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants