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

Edited contributing guidelines #1000

Merged
merged 4 commits into from Dec 9, 2022
Merged

Edited contributing guidelines #1000

merged 4 commits into from Dec 9, 2022

Conversation

hugwuoke
Copy link
Contributor

My edits were based on my experience with getting set up on GitHub. I added some minor changes of things I wish I had read while I was trying to set up my account.

  • Does this PR have a descriptive title that could go in our release notes?
  • Does this PR add any new dependencies?
  • Does this PR modify any existing APIs?
    • Is the change to the API backwards compatible?
  • Have you built the documentation (reference and/or tutorial) and verified the generated documentation is appropriate?

Reference Issues/PRs

What does this implement/fix? Briefly explain your changes.

Any other comments?

hugwuoke and others added 3 commits November 30, 2022 18:45
My edits were based on my experience with getting set up on GitHub. I added some minor changes of things I wish I had read while I was trying to set up my account.
@bdpedigo bdpedigo self-requested a review December 8, 2022 20:09
@bdpedigo bdpedigo changed the title Editing contributing guidelines Edited contributing guidelines Dec 9, 2022
@bdpedigo bdpedigo merged commit 5bc51bd into graspologic-org:dev Dec 9, 2022
j1c pushed a commit that referenced this pull request Dec 10, 2022
* Fix seaborn syntax

* Increment bugfix version

* Exclude 3.6.1

I added to line 35. I assumed the asterisk on 3.3 meant all versions of 3.3 ex (3.3.1, 3.3.2) are all excluded from the matplotlib. 

Is this correct?

* Update setup.cfg

* Update README.md

Removed outdated Zenodo DOI from README.md

* Fixed graph matching with similarity matrix of unequal dimensions (#1002)

* Fix seaborn syntax

* fixed padding of a similarity amtrix for graph matching

* fixing mypy complaint from _adj_pad

* use naive padding for similarity

* add a test

* Revert "Fix seaborn syntax"

This reverts commit fd38e05.

* try to fix ipython error in doc generation

* just make sure the original S had the right shape

* run black

* fix tests

Co-authored-by: Benjamin Pedigo <benjamindpedigo@gmail.com>

* Edited contributing guidelines (#1000)

* Editing contributing guidelines 

My edits were based on my experience with getting set up on GitHub. I added some minor changes of things I wish I had read while I was trying to set up my account.

* fix typo

Co-authored-by: Benjamin Pedigo <benjamindpedigo@gmail.com>

* Update setup.cfg (#999)

* Added release notes for 2.0.1 (#1003)

Co-authored-by: Dax Pryce <daxpryce@microsoft.com>
Co-authored-by: hugwuoke <85888975+hugwuoke@users.noreply.github.com>
Co-authored-by: Kartikeya Tripathi <96724863+ktwillcode@users.noreply.github.com>
Co-authored-by: dokato <dkk33@cam.ac.uk>
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.

None yet

2 participants