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

about parameter #3

Closed
stevexxs opened this issue Nov 18, 2019 · 1 comment
Closed

about parameter #3

stevexxs opened this issue Nov 18, 2019 · 1 comment

Comments

@stevexxs
Copy link

stevexxs commented Nov 18, 2019

Thanks for your good work.

But from parameter file which could reproduce your work,

Adv-ALSTM:
    python pred_lstm.py -l 5 -u 4 -l2 1 -v 1 -rl 1 -q ./saved_model/acl18_alstm/exp -la 0.01 -le 0.05

Adv-ALSTM:
    python pred_lstm.py -p ./data/kdd17/ourpped/ -l 15 -u 16 -l2 0.001 -v 1 -rl 1 -q ./saved_model/kdd17_alstm/model -la 0.05 -le 0.001 -f 1

seems attention("-a") and hinge loss("-hi") is not used, which is a key improvement points of your paper,
could you please tell me the reason ? Thanks !

@stevexxs
Copy link
Author

Sorry, I saw default value in parser.add_argument is 1.

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

1 participant