Skip to content

Add note to prevent using multiple time functions in a single ML job #1287

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

Open
sodhikirti07 opened this issue Apr 28, 2025 · 1 comment
Open
Assignees
Labels
Team:Platform Issues owned by the Platform Docs Team

Comments

@sodhikirti07
Copy link

Description

The Security ML team during testing of one of our Anomaly detection job found out that using both time_of_week and time_of_day functions within a single job config is not ideal. This is because time_of_week uses a different time format, modeling time as epoch seconds modulo the number of seconds in a week. To prevent confusion and save time, it’s recommended to add a note advising against this, either at the beginning of the page or within the time_of_week section.

We can add something similar to below:

Since time_of_day and time_of_week use different time formats, we recommend against using them together in a single Anomaly Detection job.

Related Issue:

@github-actions github-actions bot added the needs-team Issues pending triage by the Docs Team label Apr 28, 2025
@sodhikirti07 sodhikirti07 added the Team:Platform Issues owned by the Platform Docs Team label Apr 29, 2025
@github-actions github-actions bot removed the needs-team Issues pending triage by the Docs Team label Apr 29, 2025
@jmikell821
Copy link
Contributor

Hi @szabosteve -- can you take a look at this issue? Let me know if we need to do anything on the Security side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Platform Issues owned by the Platform Docs Team
Projects
None yet
Development

No branches or pull requests

3 participants