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

Bi-rnn's implementation #7

Open
thaddywu opened this issue Sep 3, 2021 · 1 comment
Open

Bi-rnn's implementation #7

thaddywu opened this issue Sep 3, 2021 · 1 comment

Comments

@thaddywu
Copy link

thaddywu commented Sep 3, 2021

Hi, how's it going! So much thanks for your previous sincere and prompt reply to my last issue! I hope you enjoy your day~

Here's one thing I wanna ask. Due to your paper, you guys implemented Bi-RNN following the same setup described in usenix15. But the input is a 256-dimensional one-hot vector in their paper, while birnn.py take the single byte directly as the input to the ML model. Your paper reports that Bi-RNN fails to achieve high accuracy on SPEC datasets, accounting for <80% F1-score. I modified your code to the one-hot vector input version, but found the F1 score could quickly rise up to 90%+ after several epochs, inconsistent to your results.

I'm curious about how did you split train-test datasets and calculate its overlap rate? Though this part has been mentioned in
your paper, could you make it clearer please? And I'm wondering on which setting your report of Bi-RNN is based, the single-byte input version, or the one-hot vector version. :)

Hopy you have a good weekend.

@peikexin9
Copy link
Member

Hi @thaddywu, Thanks for your interest! Could you share your code (including how you generate the data) so I can take a look? We can also have a meeting if you want :-)

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