Refactor flowchart models for modular pipelines change

[ravi-kumar-pilla]

Description

Partially resolves - #1899

Development notes

• Removes namespace property from GraphNode and retain the property only for TaskNode following Kedro Framework
• The data type of modular pipelines is changed to Optional[Set(str)]
• Updated all the create_node class methods like (create_task_node, create_data_node, create_parameters_node etc) to accept modular_pipelines parameter as we are populating modular pipelines before creating the nodes and associating each node with the associated modular pipeline during node creation.
• Remove internal_inputs/outputs and external_inputs/outputs from ModularPipelineNode. We will only have inputs/outputs for a ModularPipelineNode (i.e., the datasets which are free inputs/outputs as per Kedro)
• Update tests

QA notes

• This PR is part of a bigger refactor (https://github.com/kedro-org/kedro-viz/pull/1897) of modular pipelines and is created to ease review process.
• The CI build might fail as this PR is not self-sufficient

Checklist

• [x] 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 new entries to the RELEASE.md file
• [x] Added tests to cover my changes

[ravi-kumar-pilla]

In the description you mention

Removes namespace property from GraphNode and retain the property only for TaskNode following Kedro Framework

and I see a lot of methods related to namespaces have been removed. Can you explain why that change was needed and how namespaces are now set and used in Viz?

So earlier we used to have namespaces at the GraphNode level (which includes TaskNode, DataNode and others). After having discussion with Ivan, we realized datasets were never a part of a namespace and to have the same schema as Kedro, we removed the namespace from GraphNode and retained it to TaskNode.

For a TaskNode namespace is set as before via kedro object like -

    @field_validator("namespace")
    @classmethod
    def set_namespace(cls, _, info: ValidationInfo):
        return info.data["kedro_obj"].namespace

For other nodes, we do not need namespace. Lot of methods related to namespace like _expand_namespaces, _get_namespace in GraphNode are not required as they were used to calculate the modular_pipelines each node belongs to . Since we shifted the logic ( as you mentioned here ) before creating the nodes, we do not need them now.

Thank you