Skip to content

Commit

Permalink
Merge branch 'main' into release/10.1.0
Browse files Browse the repository at this point in the history
Signed-off-by: Jitendra Gundaniya <[email protected]>
  • Loading branch information
jitu5 committed Nov 20, 2024
2 parents 579b0b3 + 40dd06a commit 579b04d
Show file tree
Hide file tree
Showing 13 changed files with 47 additions and 24 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/all-checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ jobs:
strategy:
matrix:
os: [ windows-latest, ubuntu-latest ]
python-version: [ "3.9", "3.10", "3.11", "3.12" ]
python-version: [ "3.9", "3.10", "3.11", "3.12", "3.13" ]
uses: ./.github/workflows/e2e-tests.yml
with:
os: ${{ matrix.os }}
Expand All @@ -24,7 +24,7 @@ jobs:
strategy:
matrix:
os: [ windows-latest, ubuntu-latest ]
python-version: [ "3.9", "3.10", "3.11", "3.12" ]
python-version: [ "3.9", "3.10", "3.11", "3.12", "3.13" ]
uses: ./.github/workflows/unit-tests.yml
with:
os: ${{ matrix.os }}
Expand All @@ -34,7 +34,7 @@ jobs:
strategy:
matrix:
os: [ ubuntu-latest ]
python-version: [ "3.9", "3.10", "3.11", "3.12" ]
python-version: [ "3.9", "3.10", "3.11", "3.12", "3.13" ]
uses: ./.github/workflows/lint.yml
with:
os: ${{ matrix.os }}
Expand Down
6 changes: 3 additions & 3 deletions .github/workflows/build-backend.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ jobs:
strategy:
matrix:
os: [ windows-latest, ubuntu-latest ]
python-version: [ "3.9", "3.10", "3.11", "3.12" ]
python-version: [ "3.9", "3.10", "3.11", "3.12", "3.13" ]
uses: ./.github/workflows/e2e-tests.yml
with:
os: ${{ matrix.os }}
Expand All @@ -27,7 +27,7 @@ jobs:
strategy:
matrix:
os: [ windows-latest, ubuntu-latest ]
python-version: [ "3.9", "3.10", "3.11", "3.12" ]
python-version: [ "3.9", "3.10", "3.11", "3.12", "3.13" ]
uses: ./.github/workflows/unit-tests.yml
with:
os: ${{ matrix.os }}
Expand All @@ -37,7 +37,7 @@ jobs:
strategy:
matrix:
os: [ ubuntu-latest ]
python-version: [ "3.9", "3.10", "3.11", "3.12" ]
python-version: [ "3.9", "3.10", "3.11", "3.12", "3.13" ]
uses: ./.github/workflows/lint.yml
with:
os: ${{ matrix.os }}
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/docs-only-checks.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ jobs:
strategy:
matrix:
os: [ ubuntu-latest ]
python-version: ["3.9", "3.10", "3.11", "3.12" ]
python-version: ["3.9", "3.10", "3.11", "3.12", "3.13" ]
uses: ./.github/workflows/lint.yml
with:
os: ${{ matrix.os }}
Expand Down
8 changes: 7 additions & 1 deletion RELEASE.md
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ Please follow the established format:

## Major features and improvements

- Add support for Python 3.12
- Add support for Python 3.12, 3.13 (#2190, #2191)
- Update Kedro-Viz telemetry for opt-out model (#2022)
- Introduce `behaviour` prop object with `reFocus` prop (#2161)

Expand All @@ -25,8 +25,14 @@ Please follow the established format:
- Refactor `node-list-tree` component. (#2193)
- Fix 404 error when accessing the experiment tracking page on the demo site. (#2179)
- Check port availability before starting Kedro Viz to prevent unintended redirects when the port is in use. (#2176)
- Update kedro-viz lite banner icon and message. (#2196)
- Include Kedro Viz version in telemetry. (#2194)

## Community contributions

Many thanks to the following Kedroids for contributing PR to this release:

Check warning on line 33 in RELEASE.md

View workflow job for this annotation

GitHub Actions / vale

[vale] RELEASE.md#L33

[Kedro-viz.weaselwords] 'Many' is a weasel word!
Raw output
{"message": "[Kedro-viz.weaselwords] 'Many' is a weasel word!", "location": {"path": "RELEASE.md", "range": {"start": {"line": 33, "column": 1}}}, "severity": "WARNING"}

Check warning on line 33 in RELEASE.md

View workflow job for this annotation

GitHub Actions / vale

[vale] RELEASE.md#L33

[Kedro-viz.Spellings] Did you really mean 'Kedroids'?
Raw output
{"message": "[Kedro-viz.Spellings] Did you really mean 'Kedroids'?", "location": {"path": "RELEASE.md", "range": {"start": {"line": 33, "column": 30}}}, "severity": "WARNING"}
* [Yury Fedotov](https://github.com/yury-fedotov)

Check warning on line 34 in RELEASE.md

View workflow job for this annotation

GitHub Actions / vale

[vale] RELEASE.md#L34

[Kedro-viz.Spellings] Did you really mean 'Yury'?
Raw output
{"message": "[Kedro-viz.Spellings] Did you really mean 'Yury'?", "location": {"path": "RELEASE.md", "range": {"start": {"line": 34, "column": 4}}}, "severity": "WARNING"}

Check warning on line 34 in RELEASE.md

View workflow job for this annotation

GitHub Actions / vale

[vale] RELEASE.md#L34

[Kedro-viz.Spellings] Did you really mean 'Fedotov'?
Raw output
{"message": "[Kedro-viz.Spellings] Did you really mean 'Fedotov'?", "location": {"path": "RELEASE.md", "range": {"start": {"line": 34, "column": 9}}}, "severity": "WARNING"}


# Release 10.0.0

Expand Down
4 changes: 2 additions & 2 deletions cypress/tests/ui/flowchart/banners.cy.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,8 +17,8 @@ describe('Banners in Kedro-Viz', () => {

// Assert after action
cy.get('[data-test="flowchart-wrapper--lite-banner"]').should('exist');
cy.get('.banner-message-body').should('contains.text', 'please install the missing Kedro project dependencies')
cy.get('.banner-message-title').should('contains.text', 'Missing dependencies')
cy.get('.banner-message-body').should('contains.text', 'Some features might be disabled in --lite mode due to missing dependencies')
cy.get('.banner-message-title').should('contains.text', 'Lite mode enabled')

// Test Learn more link
cy.get(".banner a")
Expand Down
2 changes: 1 addition & 1 deletion demo-project/src/demo_project/requirements.in
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ pytest-mock>=1.7.1, <2.0
pytest~=6.2
scikit-learn~=1.0
wheel>=0.35, <0.37
pillow~=9.0
pillow>=9.0
matplotlib==3.5.0
pre-commit~=1.17
ruff==0.7.0
Expand Down
2 changes: 1 addition & 1 deletion demo-project/src/docker_requirements.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
kedro>=0.18.0
kedro-datasets[pandas.CSVDataset,pandas.ExcelDataset, pandas.ParquetDataset, plotly.PlotlyDataset, matplotlib.MatplotlibWriter]>=2.1.0
scikit-learn~=1.0
pillow~=9.0
pillow>=9.0
seaborn>=0.13.0
15 changes: 10 additions & 5 deletions package/features/steps/lower_requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -9,13 +9,18 @@ plotly==4.8
packaging==23.0
pandas==1.3; python_version < '3.10'
pandas==1.5; python_version >= '3.10' and python_version < '3.12'
pandas==2.1.1; python_version >= '3.12'
pandas==2.1.1; python_version >= '3.12' and python_version < '3.13'
pandas==2.2.3; python_version >= '3.13'
sqlalchemy==1.4
strawberry-graphql==0.192.0
networkx==2.5
orjson==3.9
strawberry-graphql==0.192.0; python_version < '3.13'
strawberry-graphql==0.240.1; python_version >= '3.13'
networkx==2.5; python_version < '3.13'
networkx==2.7; python_version >= '3.13'
orjson==3.9; python_version < '3.13'
orjson==3.10.10; python_version >= '3.13'
secure==0.3.0
# numpy 2.0 breaks with old versions of pandas and this
# could be removed when the lowest version supported is updated
numpy==1.26.4
numpy==1.26.4; python_version < '3.13'
numpy==2.1.0; python_version >= '3.13'
pathspec==0.12.1
2 changes: 1 addition & 1 deletion package/kedro_viz/__init__.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ class KedroVizPythonVersionWarning(UserWarning):
"""Custom class for warnings about incompatibilities with Python versions."""


if sys.version_info >= (3, 13):
if sys.version_info >= (3, 14):
warnings.warn(
"""Please be advised that Kedro Viz is not yet fully
compatible with the Python version you are currently using.""",
Expand Down
4 changes: 2 additions & 2 deletions package/tests/test_import.py
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,8 @@


def test_import_kedro_viz_with_no_official_support_emits_warning(mocker):
"""Test importing kedro Viz with python>=3.13 and controlled warnings should work"""
mocker.patch("kedro_viz.sys.version_info", (3, 13))
"""Test importing kedro Viz with python>=3.14 and controlled warnings should work"""
mocker.patch("kedro_viz.sys.version_info", (3, 14))

# We use the parent class to avoid issues with `exec_module`
with pytest.warns(UserWarning) as record:
Expand Down
4 changes: 2 additions & 2 deletions src/components/flowchart-wrapper/flowchart-wrapper.js
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ import ExportModal from '../export-modal';
import FlowChart from '../flowchart';
import PipelineWarning from '../pipeline-warning';
import LoadingIcon from '../icons/loading';
import AlertIcon from '../icons/alert';
import InfoBannerIcon from '../icons/info-banner';
import MetaData from '../metadata';
import MetadataModal from '../metadata-modal';
import ShareableUrlMetadata from '../shareable-url-modal/shareable-url-metadata';
Expand Down Expand Up @@ -342,7 +342,7 @@ export const FlowChartWrapper = ({
{displayMetadataPanel && <MetaData />}
{showBanner(BANNER_KEYS.LITE) && (
<Banner
icon={<AlertIcon />}
icon={<InfoBannerIcon />}
message={{
title: BANNER_METADATA.liteModeWarning.title,
body: BANNER_METADATA.liteModeWarning.body,
Expand Down
12 changes: 12 additions & 0 deletions src/components/icons/info-banner.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions src/config.js
Original file line number Diff line number Diff line change
Expand Up @@ -214,8 +214,8 @@ export const NODE_TYPES = {

export const BANNER_METADATA = {
liteModeWarning: {
title: 'Missing dependencies',
body: 'For the best experience with full functionality, please install the missing Kedro project dependencies.',
title: 'Lite mode enabled',
body: 'Some features might be disabled in --lite mode due to missing dependencies. You can find more information about lite mode in our docs.',
docsLink:
'https://docs.kedro.org/projects/kedro-viz/en/latest/kedro-viz_visualisation.html#visualise-a-kedro-project-without-installing-project-dependencies',
},
Expand Down

0 comments on commit 579b04d

Please sign in to comment.