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

mis-match between the paper and the repo. #9

Open
MjdMahasneh opened this issue Jul 30, 2021 · 3 comments
Open

mis-match between the paper and the repo. #9

MjdMahasneh opened this issue Jul 30, 2021 · 3 comments

Comments

@MjdMahasneh
Copy link

Hello,

Thank you for sharing your work. I have noticed a mis-match between the paper and the repo:

1-in the paper, you mention using RELU, while in the code you're using LeakyRELU.
2-in the paper, you mention using SGD, while in the code you're using ADAM.
3-in the paper, you mention using a decaying learning rate (by a factor of 2.5% for each epoch), while in the code you're using a static learning rate.

Could you kindly elaborate on which settings to use?

Many thanks :)

@torki-hossein
Copy link

Hi, any reason for this Q?
#9 (comment)

@MjdMahasneh
Copy link
Author

@torki-hossein Yes, I am actually trying to reproduce this work for comparative purposes.

@torki-hossein
Copy link

@torki-hossein Yes, I am actually trying to reproduce this work for comparative purposes.

I'm not the author of this work, and I am also trying to reproduce this work for comparative and extending purposes. please answer my mail (in your mailbox). ;)

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

2 participants