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

Suggestion: play_gnm and play_gnp #152

Closed
szhorvat opened this issue Apr 4, 2022 · 4 comments
Closed

Suggestion: play_gnm and play_gnp #152

szhorvat opened this issue Apr 4, 2022 · 4 comments
Labels
feature a feature request or enhancement

Comments

@szhorvat
Copy link

szhorvat commented Apr 4, 2022

This is just a small suggestion to separate play_erdos_renyi into gnm and gnp versions. This is what the modern interface does as well in igraph, see sample_gnm and sample_gnp (erdos.renyi.game is deprecated). Mixing up the two in a single function is not particularly productive. Also, while today both the G(n,p) and G(n,m) are sometimes referred to as the Erdős-Rényi model, the original paper of Erdős and Rényi actually considered G(n,m), not G(n,p).

@thomasp85 thomasp85 added the feature a feature request or enhancement label Nov 2, 2023
@szhorvat
Copy link
Author

szhorvat commented Nov 6, 2023

Thanks! It's good that this was done, as in a future version of igraph, the parameters of sample_gnm() and sample_gnp() will diverge (the former will get an extra parameter, but not the latter).

@szhorvat
Copy link
Author

szhorvat commented Nov 6, 2023

Also, just in case you are preparing a new release, be aware that a new version of igraph is coming very soon, with some deprecations and small breaking changes (no breakage revealed in tidygraph yet due to this but you never know...) You might want to wait for that release.

@thomasp85
Copy link
Owner

Thanks for the heads up - do you know the timeframe of the igraph release?

@szhorvat
Copy link
Author

szhorvat commented Nov 6, 2023

@krlmlr would know.

@szhorvat szhorvat mentioned this issue Nov 10, 2023
21 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature a feature request or enhancement
Projects
None yet
Development

No branches or pull requests

2 participants