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

Concept requirement for load_vertices and load_edges fail in implementation #42

Open
pratzl opened this issue Oct 29, 2023 · 1 comment
Assignees

Comments

@pratzl
Copy link
Contributor

pratzl commented Oct 29, 2023

load_vertices() has requires copyable_vertex<invoke_result<VProj, ranges::range_value_t>, VId, VV> and load_edges() has a similar requirement, but callers fail when enabled.

Using them as-is for the paper is sufficient for now, but we need to track that down. It may be in subtleties, like moving a string name from the original vector into the graph.

@pratzl pratzl self-assigned this Oct 29, 2023
@pratzl
Copy link
Contributor Author

pratzl commented Oct 29, 2023

The implementation has both a const VRng& and VRng&& versions. The const version may be sufficient.

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