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

Pretty printing: DataCatalog.__repr__ #3981

Closed
ElenaKhaustova opened this issue Jul 2, 2024 · 3 comments
Closed

Pretty printing: DataCatalog.__repr__ #3981

ElenaKhaustova opened this issue Jul 2, 2024 · 3 comments
Labels
Issue: Feature Request New feature or improvement to existing feature

Comments

@ElenaKhaustova
Copy link
Contributor

Description

Parent ticket: #3913
Blocked by: #3980

Implement __repr__ for DataCatalog for better catalog representation and printing.

Context

#3913 (comment)
#1721

Possible Implementation

@ElenaKhaustova ElenaKhaustova added the Issue: Feature Request New feature or improvement to existing feature label Jul 2, 2024
@ElenaKhaustova
Copy link
Contributor Author

ElenaKhaustova commented Jul 4, 2024

Similarly to printing datasets two approaches were implemented:

  • both approaches implement __repr__() method for _FrozenDatasets and DataCatalog based on the implementation __repr__() for dataset with aadded top-level keys - datastets' names. Since __str__ method is not implemented __repr__() is called when converting object to string, so we have the same results when print(obj), obj.
  • Pretty printing catalog #3990

Screenshot 2024-07-05 at 00 31 48
Screenshot 2024-07-05 at 00 32 10

@ElenaKhaustova
Copy link
Contributor Author

The non-indented version was preferred to the indented, closing #3992

@ElenaKhaustova
Copy link
Contributor Author

Solved in #3990

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue: Feature Request New feature or improvement to existing feature
Projects
Status: Done
Development

No branches or pull requests

1 participant