-
Notifications
You must be signed in to change notification settings - Fork 51
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
train_inst2vec.py fails on specific file during vocabulary building #11
Comments
The file, remaining line is
|
Removing this file didn't solve the problem for me, perhaps a memory of this file is stored somewhere else (other than in the *_preprocessing folder?) |
With a fresh checkout, I got this error in a different place:
|
@Zacharias030 Is this fixed? |
hi do you solve the problem? |
I met this problem, too. And removing all SobolPrimitives.* files even with "data_preprocessed_pickle" in the folder of "amd_preprocessed" seems to doesn't work. Do you have any solutions? |
I try to solve this problem by adding few lines in inst2vec_vocabulary.py at line 632:
it seems to have worked, but I don't know whether it will have influence in the next several steps. |
@island255 interesting, thanks. It looks like something that would work around this issue, but I'm wondering which files create empty graphs. |
Co-authored-by: island255 <island255@users.noreply.github.com>
@island255 @Marvinmw can you please check if PR #25 works for you? |
Sure, it works. |
Skip empty dual graphs (Fixes Issue #11)
Closing the issue since the problem is fixed. Please write here if I should reopen it. |
Workaround: delete
ncc/data/shoc/sho/ProgressBar.ll
from the dataset before preprocessing.The file is
ncc/data/shoc/sho/ProgressBar.ll
and the problem seems to be that this file has only a single LLVM IR instruction that goes into the XFG and later we end up with an empty graph and a call tobuild_H_dictionary(D, skip_window, folder, filename, dictionary, stmts_cut_off)
that fails at
nx.adjacency_matrix(D)
.The xfg has 2 nodes: a
root_node
and the one mentioned above, so IMHO the dual-xfg should still contain a single node, which it doesn't and that is causing the problem.The text was updated successfully, but these errors were encountered: