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

Fix handling of save_show_or_return flags #414

Merged
merged 10 commits into from
May 4, 2023

Conversation

elfofmaxwell
Copy link
Collaborator

Fix #326

The following APIs are modified to accept 'save', 'show', 'return', 'both', and 'all':

  • pl.cell_cycle_scores
  • pl.connectivity_base
  • pl.nneighbors
  • pl.phase_portraits
  • pl.dynamics
  • pl.fate_bias
  • pl.fate
  • pl.least_action
  • pl.bubble
  • pl.nxvizPlot
  • pl.arcPlot
  • pl.hivePlot
  • pl.basic_stats
  • pl.show_fraction
  • pl.variance_explained
  • pl.biplot
  • pl.loading
  • pl.feature_genes
  • pl.exp_by_groups
  • pl.plot_direct_graph
  • pl.scatters
  • pl.show_landscape
  • pl.cell_wise_vectors_3d
  • pl.line_integral_conv
  • pl.cell_wise_vectors
  • pl.grid_vectors
  • pl.streamline_plot
  • pl.plot_energy
  • pl.state_graph
  • pl.plot_3d_streamtube
  • pl.kinetic_curves
  • pl.kinetic_heatmap
  • pl.jacobian_kinetics acceptable value updated from save_fig to save
  • pl.sensitivity_kinetics acceptable value updated from save_fig to save
  • pl.plot_flow_field
  • pl.plot_nullclines
  • pl.plot_fixed_points_2d
  • pl.plot_fixed_points
  • pl.plot_traj
  • pl.plot_separatrix
  • pl.topography
  • pl.jacobian
  • pl.jacobian_heatmap
  • pl.sensitivity
  • pl.sensitivity_heatmap

The following APIs now only accept 'save', 'show', 'both' and would support 'return', 'all' in future:

  • pl.response
  • pl.plot_hill_function
  • pl.causality
  • pl.comb_logic
  • pl.hessian
  • pl.lap_min_time

The following APIs rely on previously mentioned APIs and accept 'save', 'show', 'return', 'both', and 'all' as kwargs:

  • pl.infomap
  • pl.leiden
  • pl.louvain
  • pl.pca
  • pl.trimap
  • pl.tsne
  • pl.umap
  • pl.space
  • pl.acceleration
  • pl.curl
  • pl.curvature
  • pl.divergence
  • pl.speed

elfofmaxwell and others added 9 commits September 2, 2022 19:01
heatmaps, and least_action_path
Update black version
- least_action_path.lap_min_time
- heatmaps.response
- heatmaps.causality
- least_action_path.lap_min_time
- heatmaps.response
- heatmaps.causality
- heatmaps.plot_hill_function
in cell_cycle, connectivity, dynamics
@Xiaojieqiu Xiaojieqiu self-requested a review February 23, 2023 21:21
@Sichao25
Copy link
Collaborator

Sichao25 commented Mar 9, 2023

This pull request looks good to me.

Copy link
Contributor

@Ukyeon Ukyeon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Xiaojieqiu
Please review.

@Xiaojieqiu Xiaojieqiu merged commit 30207b9 into aristoteleo:master May 4, 2023
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

Successfully merging this pull request may close these issues.

matplotlib 3.5.1 compatibiliy known issue collection
4 participants