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: Implementation of a Class Incremental Learning Algorithm Evaluation System based on Ianvs #85

Merged
merged 2 commits into from
Oct 31, 2023

Conversation

qxygxt
Copy link
Contributor

@qxygxt qxygxt commented Oct 31, 2023

No description provided.

@qxygxt qxygxt force-pushed the main branch 2 times, most recently from 507dfce to b7039df Compare October 31, 2023 02:22
@Frank-lilinjie
Copy link
Contributor

lgtm

@@ -0,0 +1,38 @@
for i in range(len(nam_label)):
Copy link
Member

Choose a reason for hiding this comment

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

If this is not a useful file, please remove it in this commit.

'best_pred': self.trainer.best_pred,
}, is_best)

# if not self.trainer.args.no_val and \
Copy link
Member

Choose a reason for hiding this comment

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

If this is unuseful debug lines, it should not be committed.

img /= self.std

# mean and std for original depth images
mean_depth = 0.12176
Copy link
Member

Choose a reason for hiding this comment

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

what is these magic number meaning? It is suggested to add comment for these number.

depth = sample['depth']
mask = sample['label']
width, height = img.size
left = 140
Copy link
Member

Choose a reason for hiding this comment

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

what is these magic number meaning? It is suggested to add comment for these number.

If these numbers are not the only value of varibles, please config these number in a seperated CONSTANT file.

img = img.resize((width,height), Image.BILINEAR)
depth = depth.resize((width,height), Image.BILINEAR)
mask = mask.resize((width,height), Image.NEAREST)
# img = img.resize((512,1024), Image.BILINEAR)
Copy link
Member

Choose a reason for hiding this comment

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

Suggest to remove these debug lines, Also the following.

@@ -0,0 +1,50 @@
import os
os.environ['BACKEND_TYPE'] = 'PYTORCH'
Copy link
Member

Choose a reason for hiding this comment

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

remove unused lines

import os

os.environ['BACKEND_TYPE'] = 'PYTORCH'
# os.environ["UNSEEN_SAVE_URL"] = "s3://kubeedge/sedna-robo/unseen_samples/"
Copy link
Member

Choose a reason for hiding this comment

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

please remove unused lines

self.eval_interval = kwargs.get("eval_interval", 50)
self.no_val = kwargs.get("no_val", True)
self.cuda = True
self.savedir = '/home/QXY/dataset/save'
Copy link
Member

Choose a reason for hiding this comment

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

Hardcoded path should not be committed here, suggest to use the relative path.

idp_obstacle = (x_onehot[:, 19] - tpmult[:, 19]).sum()
tp_nonobstacle = (-1*y_onehot+1).sum()

# for i in range(0, x.size(0)):
Copy link
Member

Choose a reason for hiding this comment

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

unused lines should be removed.

@jaypume
Copy link
Member

jaypume commented Oct 31, 2023

/lgtm

@jaypume
Copy link
Member

jaypume commented Oct 31, 2023

/approve

@jaypume jaypume merged commit 6e8a115 into kubeedge:main Oct 31, 2023
9 checks passed
@MooreZheng
Copy link
Collaborator

MooreZheng commented May 31, 2024

Related issue and PR: #79 , #82

wyoung1 pushed a commit to wyoung1/ianvs that referenced this pull request Sep 25, 2024
OSPP: Implementation of a Class Incremental Learning Algorithm Evaluation System based on 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.

4 participants