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

[OSPP] The codes of Unknown Task Recognition Algorithm Reproduction based on Lifelong Learning of Ianvs #42

Merged

Conversation

Frank-lilinjie
Copy link
Contributor

@Frank-lilinjie Frank-lilinjie commented Oct 24, 2022

The PR of the proposal is in PR

This PR is merged in the branch named feature-lifelong-n.

This PR is about the codes of Unknown Task Recognition Algorithm Reproduction based on Lifelong Learning of Ianvs.

This PR is a modification of the last PR

transforms.Resize([32,32]),
transforms.ToTensor()
])
#import pdb
Copy link
Contributor

Choose a reason for hiding this comment

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

Annotated codes should be removed.

#pdb.set_trace()
InferenceData=data_transforms(Image.open(x_data[0][0]))
device = 'cuda' if torch.cuda.is_available() else 'cpu'
pthfile = "./examples/curb-detection/lifelong_learning_bench/testalgorithms/rfnet/results/Epochofprose17.pth"
Copy link
Contributor

@luosiqi luosiqi Oct 25, 2022

Choose a reason for hiding this comment

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

The value of "pthfile" should be assigned in init function by a passed parameter.

@ClassFactory.register(ClassType.UTD, alias="UnseenSampleRecognitionByScene")
class UnseenSampleRecognitionByScene:
def __init__(self, **kwargs):
self.model_path = kwargs.get("model_path")
Copy link
Contributor

Choose a reason for hiding this comment

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

It seems that this parameter is not utilized.

__all__ = ('UnseenSampleRecognitionByScene',)

@ClassFactory.register(ClassType.UTD, alias="UnseenSampleRecognitionByScene")
class UnseenSampleRecognitionByScene:
Copy link
Contributor

Choose a reason for hiding this comment

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

Add documentation comments to make this class more clarified.

hyperparameters:
- model_path:
values:
- "./examples/curb-detection/lifelong_learning_bench/testalgorithms/rfnet/models/Epochofprose17.pth"
Copy link
Contributor

Choose a reason for hiding this comment

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

The model file "Epochofprose17.pth" has no origin which should be provided in readme.

@kubeedge-bot
Copy link
Collaborator

@Frank-lilinjie: PR needs rebase.

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
Collaborator

The code is overall fine but the conflict need to be resolved by fetching the latest version

@@ -0,0 +1,48 @@
from typing import List, Any, Tuple
Copy link
Collaborator

Choose a reason for hiding this comment

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

The origin reproduced paper should be referred by comment at the beginning of the code

…orithm

update algorithms
delete annotated codes
refer the origin reproduced p
add class documentation
upload Readme
Modify folder name
add an example
modify readme
Supplementary curb-detection cases

Signed-off-by: Frank-lilinjie <lilinjie@bupt.edu.cn>
@jaypume
Copy link
Member

jaypume commented Oct 31, 2022

/lgtm

@luosiqi
Copy link
Contributor

luosiqi commented Oct 31, 2022

lgtm

@jaypume
Copy link
Member

jaypume commented Oct 31, 2022

/approve

@jaypume jaypume merged commit 883ce0e into kubeedge:feature-lifelong-n Oct 31, 2022
wyoung1 pushed a commit to wyoung1/ianvs that referenced this pull request Sep 25, 2024
[OSPP] The codes of Unknown Task Recognition Algorithm Reproduction based on Lifelong Learning of Ianvs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants