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

FEA add plot_contingency_matrix / ContingencyMatrixDisplay #19147

Closed
wants to merge 9 commits into from

Conversation

yash-clear
Copy link

Reference Issues/PRs

#19107

What does this implement/fix? Explain your changes.

Adding contingency_matrix_plot

Any other comments?

cmarmo and others added 3 commits December 2, 2020 17:22
Co-authored-by: Juan Carlos Alfaro Jiménez <JuanCarlos.Alfaro@uclm.es>
Co-authored-by: Leonardo Uieda <leouieda@gmail.com>
Co-authored-by: Nadim Kawwa <40652202+NadimKawwa@users.noreply.github.com>
Co-authored-by: Olivier Grisel <olivier.grisel@gmail.com>
Co-authored-by: Adrin Jalali <adrin.jalali@gmail.com>
Co-authored-by: Chiara Marmo <cmarmo@users.noreply.github.com>
Co-authored-by: Juan Carlos Alfaro Jiménez <JuanCarlos.Alfaro@uclm.es>
Co-authored-by: Matthew Calcote <matthew.calcote@connexta.com>
Co-authored-by: novaya <51009628+novaya@users.noreply.github.com>
Co-authored-by: David Katz <41651296+DavidKatz-il@users.noreply.github.com>
Co-authored-by: Frans Larsson <frans.larsson@gmail.com>
Co-authored-by: Maxime Prieur <maxime.prieur@insa-rouen.fr>
Co-authored-by: Thomas J. Fan <thomasjpfan@gmail.com>
Co-authored-by: Neal Fultz <nfultz@gmail.com>
Co-authored-by: RamyaNP <56212418+RamyaNP@users.noreply.github.com>
Co-authored-by: Thomas A Caswell <tcaswell@gmail.com>
Co-authored-by: Amol Deshmukh <34318357+des137@users.noreply.github.com>
Co-authored-by: Kot271828 <48412793+Kot271828@users.noreply.github.com>
Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com>
Co-authored-by: Nicolas Hug <contact@nicolas-hug.com>
Co-authored-by: Rohan Paul <rohan-paul@users.noreply.github.com>
Co-authored-by: Tim Gates <tim.gates@iress.com>
@NicolasHug
Copy link
Member

NicolasHug commented Jan 11, 2021

Looks like you based your changes off of a very old branch @yash-clear .
Would you mind updating it? Probably with git pull upstream master if you followed our contributing instructions (otherwise the branch and remote names might differ).

@glemaitre glemaitre changed the title Adding a new feature FEA add plot_contingency_matrix / ContingencyMatrixDisplay Jan 11, 2021
@yash-clear
Copy link
Author

I accidently committed on master branch , I want to do it on 0.24.0 how can I change it??

@jnothman
Copy link
Member

You should base it on master, but a recent version of master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants