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.

Sign up for GitHub

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

Jump to bottom

Encode time variable with sine or cosine transformation #140

Open
javiarrobas opened this issue Nov 28, 2023 · 2 comments
Open

Encode time variable with sine or cosine transformation #140

javiarrobas opened this issue Nov 28, 2023 · 2 comments
Assignees
@IamAniket12

Comments

@javiarrobas
Copy link
Contributor

Currently we're passing absolute time as an observation to the agent. This means that we do not leverage from the systematic sequence of repetitive patterns that we find in weekly and/or daily schedules in buildings. In this sense, encoding the time variable in a sine or cosine transformation is expected to be beneficial for the agent's learning. This issue is to implement that.
@IamAniket12
Copy link
Contributor

Hi @javiarrobas, I'd like to take it on if no one is assigned to this issue. Thanks!

@javiarrobas
Copy link
Contributor Author

@IamAniket12 thanks a lot for offering. Sure thing, please go ahead and ping me once ready. I'll be happy to review your suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@IamAniket12 IamAniket12

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@javiarrobas @IamAniket12