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

Regarding SpatialEmbed project #1

Open
adi1999 opened this issue May 10, 2021 · 4 comments
Open

Regarding SpatialEmbed project #1

adi1999 opened this issue May 10, 2021 · 4 comments

Comments

@adi1999
Copy link

adi1999 commented May 10, 2021

Hey @charlotte12l , hope you're doing good! I wasn't able to open an issue on the SpatialEmbeddings repo that you've modified for multiclass exp, So wanted to ask whether you were able to obtain desired performance with you implementation which you've provided. And how did you were able to generate foreground weight that you're using in your code in the loss function.
Thank you!!

@charlotte12l
Copy link
Owner

Hi:

Thanks for your interest! I can't reproduce the paper's performance with my codes so, in the end, I gave up.

http://www.robesafe.uah.es/personal/eduardo.romera/pdfs/Romera17iv.pdf
weight_C = 1/log(1.1+p_C) , p_C = the prob of class_C

I think I generate the foreground weight using this equation. I did that project a long time ago and I've forgotten most of the things, I'm really sorry. Maybe you can follow davyneven/SpatialEmbeddings#5 . Best luck!

@jyang68qq
Copy link

jyang68qq commented May 11, 2021

@adi1999 Hi, did you successfully get the multi-class setting solved?

@adi1999
Copy link
Author

adi1999 commented May 11, 2021

@charlotte12l Thank you for your response!!

@adi1999
Copy link
Author

adi1999 commented May 11, 2021

@adi1999 Hi, did you successfully get the multi-class setting solved?

hey @jyang68qq , unfortunately not, I was experimenting to improve the performance by changing architecture and loss function, in which I was successful, but only for car instance, Was trying to evaluate performance on multiclass, but unfortunately it's to working, I'll update in case I'm able to do so!!

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

No branches or pull requests

3 participants