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

Release/0.11.1 #182

Merged
merged 24 commits into from
Nov 16, 2022
Merged

Release/0.11.1 #182

merged 24 commits into from
Nov 16, 2022

Conversation

GabrielAzevedoFerreiraQB
Copy link
Contributor

Motivation and Context

Why was this PR created?

How has this been tested?

What testing strategies have you used?

Checklist

  • Read the contributing guidelines
  • Opened this PR as a 'Draft Pull Request' if it is work-in-progress
  • Updated the documentation to reflect the code changes
  • Added a description of this change and added my name to the list of supporting contributions in the RELEASE.md file
  • Added tests to cover my changes
  • Assigned myself to the PR

Notice

  • I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

  • I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

  • I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

  • I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

qbphilip and others added 24 commits November 11, 2021 15:06
Merge master (release 0.11.0) back to dev
* updating Black version

* linting - adapting to pylint updates

* updating docs_requirements

* allowing latest pre-commit

* updating pre-comm version in test_requirements

* specifying ipython_genutils version
* Fixed set_cpd for parentless nodes that are not MultiIndex
* Updated RELEASE notes
* Added test case for setcpd() when a node has no parent

Co-authored-by: Richard Oentaryo <oentaryorj@users.noreply.github.com>
Co-authored-by: oentaryorj <oentaryorj@gmail.com>
* updating Black version

* linting - adapting to pylint updates

* updating docs_requirements

* enabling python 3.9

* allowing latest pre-commit

* allowing latest pre-commit

* testing python 3.10

* testing python 3.10 - adapting scikit learn requirements

* testing python 3.10 - changing setup.py

* Revert "testing python 3.10 - changing setup.py"

This reverts commit 587af39.

* Revert "testing python 3.10 - adapting scikit learn requirements"

This reverts commit 4cd186b.

* Revert "testing python 3.10"

This reverts commit f95a463.

* updating pre-comm version in test_requirements

* updating pre-comm version in test_requirements

* specifying ipython_genutils version

* updating Readme, release and docs files

Co-authored-by: Richard Oentaryo <oentaryorj@users.noreply.github.com>
* Add dockerfiles for development on cpu and gpu
* Update instructions in CONTRIBUTING.md
* Remove scikit-learn from test_requirements.txt

Co-authored-by: Gabriel Azevedo Ferreira <57528979+GabrielAzevedoFerreiraQB@users.noreply.github.com>
Co-authored-by: oentaryorj <oentaryorj@gmail.com>
Co-authored-by: Richard Oentaryo <oentaryorj@users.noreply.github.com>
The following vulnerabilities are fixed by pinning transitive dependencies:
- https://snyk.io/vuln/SNYK-PYTHON-NBCONVERT-2979829

Co-authored-by: Richard Oentaryo <oentaryorj@users.noreply.github.com>
* Fix DAGLayer moving moving out of gpu
* Linting core.py
* Update core.py

Co-authored-by: samialab <6316945+samialab@users.noreply.github.com>
Co-authored-by: Richard Oentaryo <oentaryorj@users.noreply.github.com>
Co-authored-by: oentaryorj <oentaryorj@gmail.com>
* Update network.py

Fix for #164

* Update RELEASE.md

* update documentation and linting

* add test

* Update network.py

Missed removing a test print statement
* fix: cpd floating point comparison issue

* fix: imports ordering
* chore: update circleci

* chore: update circleci

* chore: update circleci

* chore: fix partial warnings

* chore: ignore inspect. for cov

* chore: release note

Co-authored-by: Richard Oentaryo <oentaryorj@users.noreply.github.com>
* fix: doc_requirements.txt to reduce vulnerabilities


The following vulnerabilities are fixed by pinning transitive dependencies:
- https://snyk.io/vuln/SNYK-PYTHON-NBCONVERT-2979829

* Fix: upgrade for nbconvert to 6.3.0. No python 3.6 support

* Changed nbconvert limit to <8.0

* Fix: updated nbsphinx version to 5.0 since it covers changes made in nbconvert 6+

Co-authored-by: snyk-bot <snyk-bot@snyk.io>
Co-authored-by: Paul Beaumont <33718941+pjbqb@users.noreply.github.com>
Co-authored-by: Dmitry Denisenko <Dmitry_denisenko@mckinsey.com>
* fix: doc_requirements.txt to reduce vulnerabilities


The following vulnerabilities are fixed by pinning transitive dependencies:
- https://snyk.io/vuln/SNYK-PYTHON-MISTUNE-2940625

* Fix: resolved jinja2 requirements to make mistune version bigger

* Fix: removed commentsfrom doc_requirements.txt

* Fix: reformat doc_requirements for linter

* Fix: finally corrected doc_requirements to be in alphabetical order

* Fix: added more flexibility to mistune requirements

Co-authored-by: snyk-bot <snyk-bot@snyk.io>
Co-authored-by: Richard Oentaryo <oentaryorj@users.noreply.github.com>
Co-authored-by: Dmitry Denisenko <Dmitry_denisenko@mckinsey.com>
* unblocking scipy requirements

* unblocking scipy requirements - for 3.8 onwards

* unblocking scipy requirements - for 3.8 onwards

* adding line

* removing line

Co-authored-by: Richard Oentaryo <oentaryorj@users.noreply.github.com>
* fix: requirements.txt to reduce vulnerabilities


The following vulnerabilities are fixed by pinning transitive dependencies:
- https://snyk.io/vuln/SNYK-PYTHON-WHEEL-3092128

* updating order

* updating python version for requirement

* updating python version for requirement

Co-authored-by: snyk-bot <snyk-bot@snyk.io>
Co-authored-by: GabrielAz <gabriel.azevedoferreira@quantumblack.com>
@qbphilip qbphilip changed the base branch from develop to master November 10, 2022 14:49
Copy link
Contributor

@qbphilip qbphilip left a comment

Choose a reason for hiding this comment

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

LGTM, lots of great work!

@qbphilip qbphilip merged commit d67ca0c into master Nov 16, 2022
@qbphilip qbphilip deleted the release/0.11.1 branch November 16, 2022 15:07
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.

10 participants