Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

you-n-g
Copy link
Collaborator

@you-n-g you-n-g commented Feb 26, 2021

Description

init version of online serving and rolling

Motivation and Context

Support Online Serving and Rolling

Types of changes

  • Fix bugs
  • Add new feature
  • Update documentation

@you-n-g
Copy link
Collaborator Author

you-n-g commented Apr 30, 2021

@lzh222333
#404 is merged now.

@you-n-g
Copy link
Collaborator Author

you-n-g commented Apr 30, 2021

@lzh222333

#407 has been merged now.

@Derek-Wds Derek-Wds added the enhancement New feature or request label May 10, 2021
self.logger.info(f"Strategy `{strategy.name_id}` begins routine...")
if self.status == self.STATUS_NORMAL:
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Add more documents about it.


tasks = strategy.prepare_tasks(self.cur_time, **task_kwargs)
models = self.trainer.train(tasks)
models_list.append(models)
if self.status == self.STATUS_NORMAL or not self.trainer.is_delay():
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Add more documents about it


if not delay:
if not self.trainer.is_delay():
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It will better to add a 2*2 table in this file to explain how status and delay affect the behavior of the online manager.

@you-n-g you-n-g marked this pull request as ready for review May 17, 2021 09:35
@you-n-g you-n-g merged commit d08146c into microsoft:main May 17, 2021
@you-n-g you-n-g mentioned this pull request May 23, 2021
5 tasks
qianyun210603 pushed a commit to qianyun210603/qlib that referenced this pull request Mar 23, 2023
init version of online serving and rolling
you-n-g added a commit to you-n-g/qlib that referenced this pull request Aug 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants