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.

Sign up for GitHub

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

Jump to bottom

pin ensmallen_graph to 0.3.6 for now to pass tests #348

Merged

justaddcoffee merged 1 commit into master from fix_ensmallen_error

Sep 25, 2020

Collaborator

justaddcoffee commented Sep 25, 2020

Updated ensmallen_graph causes a panic in
TestEdges.test_make_edges_pos_train_test_valid_edges_distinct_1_neg_train_edges_tsv:

        neg_train_edges, neg_test_edges = \
>           all_negative_edges.random_holdout(seed=seed, train_percentage=train_fraction)
E       pyo3_runtime.PanicException: index out of bounds: the len is 0 but the index is 53

Pinned to 0.3.6 for now


          pin ensmallen_graph to 0.3.6 for now to pass tests

d72ae27

justaddcoffee merged commit f012b45 into master

justaddcoffee deleted the fix_ensmallen_error branch

September 25, 2020 00:22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment