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

holiday parameter in calendar_td() #10

Open
AQLT opened this issue Oct 7, 2023 · 4 comments
Open

holiday parameter in calendar_td() #10

AQLT opened this issue Oct 7, 2023 · 4 comments
Assignees

Comments

@AQLT
Copy link
Contributor

AQLT commented Oct 7, 2023

The parameter holiday in calendar_td() doesn't seem to be used:
https://github.com/jdemetra/jdplus-main/blob/cb327446048d9a3300f1e6b7b6a3765a4da98757/jdplus-main-base/jdplus-toolkit-base-parent/jdplus-toolkit-base-r/src/main/java/jdplus/toolkit/base/r/modelling/Variables.java#L63-L74
Maybe this line
https://github.com/jdemetra/jdplus-main/blob/cb327446048d9a3300f1e6b7b6a3765a4da98757/jdplus-main-base/jdplus-toolkit-base-parent/jdplus-toolkit-base-r/src/main/java/jdplus/toolkit/base/r/modelling/Variables.java#L65C114-L65C130
should be replaced by something like:

HolidaysCorrectedTradingDays.HolidaysCorrector corrector = HolidaysCorrectionFactory.corrector(calendar, DayOfWeek.of(dw));
@palatej
Copy link
Contributor

palatej commented Apr 19, 2024

Hi Alain,

I finally abandoned (for the moment) the idea of specifying the day corresponding to an holiday (normally Sunday) because the correct handling of such a feature generated a lot of problems (especially in long term mean corrections). So, it is unused for the time being. A warning could be sent to the user in R (or the parameter could be suppressed).

@TanguyBarthelemy
Copy link
Contributor

Ok we will remove this parameter from the function calendar_td().

@AQLT
Copy link
Contributor Author

AQLT commented Apr 30, 2024

The parameter can still be left with a comment saying that it is unused. It would then be easier to change it when it will be implemented in Java. This is for example what is done in some set_ functions

@TanguyBarthelemy
Copy link
Contributor

Ok we will add a warning then.

@TanguyBarthelemy TanguyBarthelemy self-assigned this May 7, 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

No branches or pull requests

3 participants