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

Tutorial for ATCII Lifelong Learning Job #401

Closed
wants to merge 8 commits into from
Closed

Tutorial for ATCII Lifelong Learning Job #401

wants to merge 8 commits into from

Conversation

qxygxt
Copy link
Contributor

@qxygxt qxygxt commented Mar 24, 2023

In this PR, a tutorial based on lifelong learning job in thermal comfort prediction scenario will explain how to run the default example with customized configurations, as well as how to develop and integrate user-defined modules to users.

@kubeedge-bot
Copy link
Collaborator

Welcome @qxygxt! It looks like this is your first PR to kubeedge/sedna 🎉

@kubeedge-bot kubeedge-bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Mar 24, 2023
@qxygxt qxygxt changed the title Tutorial for ACTII lifelong learning job Tutorial for ATCII Lifelong Learning Job Mar 24, 2023
@kubeedge-bot kubeedge-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Mar 30, 2023
@MooreZheng
Copy link

MooreZheng commented Mar 31, 2023

The current version of tutorial has been review on the routine meeting of SIG AI, which looks good to me.

BTW: Sign is needed in commit.

Toturial based on lifelong learning job in thermal comfort prediction scenario to explain how to run the default example with customized configurations, as well as how to develop and integrate user-defined modules.

Signed-off-by: qxygxt <xingyu.q@outlook.com>
Signed-off-by: qxygxt <xingyu.q@outlook.com>
Signed-off-by: qxygxt <xingyu.q@outlook.com>
Signed-off-by: qxygxt <xingyu.q@outlook.com>
Signed-off-by: qxygxt <xingyu.q@outlook.com>
Signed-off-by: qxygxt <xingyu.q@outlook.com>
@kubeedge-bot kubeedge-bot added the lgtm Indicates that a PR is ready to be merged. label Mar 31, 2023

Artifacts including multi-task learning models, partitioned sample sets, etc. can be found in `outputDir`, and the inference result is stored in the `Dataset`'s url.

# 2 Develop and Integrate Customized Modules
Copy link

@MooreZheng MooreZheng Mar 31, 2023

Choose a reason for hiding this comment

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

This tutorial not only shows how to run the default example (with or without customized configurations), but also shows how to develop and integrate user-defined modules. That is indeed impressive.

@kubeedge-bot kubeedge-bot added do-not-merge/contains-merge-commits and removed lgtm Indicates that a PR is ready to be merged. labels Mar 31, 2023
@kubeedge-bot
Copy link
Collaborator

@qxygxt: Adding label do-not-merge/contains-merge-commits because PR contains merge commits, which are not allowed in this repository.
Use git rebase to reapply your commits on top of the target branch. Detailed instructions for doing so can be found here.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@MooreZheng
Copy link

/lgtm

@kubeedge-bot kubeedge-bot added the lgtm Indicates that a PR is ready to be merged. label Mar 31, 2023
@jaypume
Copy link
Member

jaypume commented Apr 20, 2023

Please squash the 7 commits to 1 commits to make the commit history more clean.

@kubeedge-bot kubeedge-bot removed the lgtm Indicates that a PR is ready to be merged. label Apr 20, 2023
@kubeedge-bot
Copy link
Collaborator

New changes are detected. LGTM label has been removed.

@kubeedge-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please ask for approval from moorezheng after the PR has been reviewed.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@qxygxt qxygxt closed this Apr 20, 2023
@qxygxt qxygxt reopened this Apr 20, 2023
@qxygxt qxygxt closed this Apr 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
do-not-merge/contains-merge-commits size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants