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

Add __str__() methods to GazeDataFrame and EventDataFrame #448

Closed
3 tasks
dkrako opened this issue Jun 9, 2023 · 0 comments · Fixed by #727
Closed
3 tasks

Add __str__() methods to GazeDataFrame and EventDataFrame #448

dkrako opened this issue Jun 9, 2023 · 0 comments · Fixed by #727
Assignees
Labels
enhancement New feature or request

Comments

@dkrako
Copy link
Contributor

dkrako commented Jun 9, 2023

Description of the problem

Currently we always need to use .frame to print the dataframes in interpreters and notebooks.
It would be better if both classes would have a nice string representation.

Description of a solution

Basically forwarding pl.DataFrame.__str__ should do the trick, but GazeDataFrame should also include the experiment definition.

Minimum acceptance criteria

  • GazeDataFrame.__str__() method which includes experiment and frame in output
  • EventDataFrame.__str__() method which includes frame in output
  • remove the usage of frame from all tutorial notebooks if it is used just for displaying the dataframe
@dkrako dkrako added the enhancement New feature or request label Jun 9, 2023
@dkrako dkrako modified the milestone: Sprint 12 Jun 9, 2023
@josephine-funken josephine-funken linked a pull request May 8, 2024 that will close this issue
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants