Skip to content

Feature: enhancements and bug fixes for network_p2p()#149

Merged
martinctc merged 11 commits into
mainfrom
feature/improve-network_p2p
Jun 17, 2021
Merged

Feature: enhancements and bug fixes for network_p2p()#149
martinctc merged 11 commits into
mainfrom
feature/improve-network_p2p

Conversation

@martinctc

@martinctc martinctc commented May 18, 2021

Copy link
Copy Markdown
Member

Summary

This branch introduces some enhancements and bug fixes to network_p2p(), as per issues mentioned on GitHub (#145, #146, #148, #159).

Changes

The changes made in this PR are:

  1. Fixed issue of legend_pos having no effect on the legend position (fixed for the 'ggraph' method). (Bug: legend positions not working for network_p2p() #148)
  2. Changed the default colour scheme of network_p2p() to light mode (Bug / Aesthetics: network_p2p() colour scheme issues #146)
  3. Added a edge_alpha argument for customization of edge transparency. This also resolves issues of edges not appearing on the console by setting the default to 1, but the argument retains the ability of the user to customize edge transparency (Bug / Aesthetics: network_p2p() colour scheme issues #146).
  4. Improved documentation to explain how to install the python dependencies for 'leiden'. (Bug: dependency error for network_p2p(display = "leiden") #145)
  5. Removed the inconsistent edge border issue when plotting using the 'ggraph' method.
  6. Added new weight argument to enable users to select custom weights for creating graph. (Feature request: ability to use different weights for network_p2p() #159)

Checks

  • All R CMD checks pass
  • roxygen2::roxygenise() has been run prior to merging to ensure that .Rd and NAMESPACE files are up to date.
  • NEWS.md has been updated.

Notes

This fixes #145, #146, #148, and #159.

@martinctc martinctc added bug Something isn't working enhancement New feature or request labels May 18, 2021
@martinctc martinctc self-assigned this May 18, 2021
Added new `edge_alpha` argument and flipped default colour scheme from black to white.
@martinctc martinctc linked an issue Jun 17, 2021 that may be closed by this pull request
@martinctc martinctc marked this pull request as ready for review June 17, 2021 09:36
@martinctc martinctc merged commit 0db513b into main Jun 17, 2021
@martinctc martinctc deleted the feature/improve-network_p2p branch June 17, 2021 09:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working enhancement New feature or request

Projects

None yet

1 participant