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

Identify network analysis functions that don't use the C core #1415

Open
6 tasks
szhorvat opened this issue Jun 21, 2024 · 4 comments
Open
6 tasks

Identify network analysis functions that don't use the C core #1415

szhorvat opened this issue Jun 21, 2024 · 4 comments

Comments

@szhorvat
Copy link
Member

szhorvat commented Jun 21, 2024

Write a list of network analysis functions that don't use the C core (such as subgraph_centrality()).

@krlmlr Can you guys help with this, so we can evaluate which of these should be moved to the C core?


List of functions

  • subgraph_centrality()
  • power_centrality()
  • alpha_centrality()
  • match_vertices()
  • layout_nicely() — the heuristics for selecting an appropriate layout could be moved to the C core
  • ... ?
@szhorvat szhorvat transferred this issue from igraph/igraph Jun 21, 2024
@krlmlr
Copy link
Contributor

krlmlr commented Jun 24, 2024

Would that be all functions that don't use *_impl() or .Call(), directly or indirectly?

I'm having issues with the flow package in this context, moodymudskipper/flow#86 and moodymudskipper/flow#175 .

CC @maelle.

@szhorvat
Copy link
Member Author

Would that be all functions that don't use *_impl() or .Call(), directly or indirectly?

Yes, that's right, but I'm only interested in network analysis functions, i.e. those which could in principle be implemented in C.

I know of at least power_centrality(), subgraph_centrality(), alpha_centrality().

@krlmlr
Copy link
Contributor

krlmlr commented Jun 24, 2024

Can you restrict what functions you are looking for in this list?

https://r.igraph.org/reference/index.html

@szhorvat
Copy link
Member Author

szhorvat commented Jul 2, 2024

@krlmlr Let's not do a full survey. If you come across something that you think might benefit from moving it to the C core, please add it to the list at the top.

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