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

for strategy in self.strategies:
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

self.history.setdefault(self.cur_time, {})[strategy] = online_models

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

4 participants