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

kedro-datasets: Update __repr__ for PartitionedDataset #769

Closed
Tracked by #778
ElenaKhaustova opened this issue Jul 18, 2024 · 0 comments
Closed
Tracked by #778

kedro-datasets: Update __repr__ for PartitionedDataset #769

ElenaKhaustova opened this issue Jul 18, 2024 · 0 comments
Assignees

Comments

@ElenaKhaustova
Copy link
Contributor

ElenaKhaustova commented Jul 18, 2024

Description

The pretty printing implementation relies on a dataset's _describe() method to get the content for printing. For PartitionedDataset _describe() returns dataset type and config which should be formatted similarly to CachedDataset for consistency.

Current representation:
Screenshot 2024-07-09 at 12 23 18

Target representation (see CachedDataset example):
Screenshot 2024-07-09 at 12 30 27

Context

kedro-org/kedro#3987 (comment)

Possible Implementation

Update PartitionedDataset.__repr__ like it's done for CachedDataset to align representations - add dataset key and use parent _pretty_repr method.

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

No branches or pull requests

1 participant