Skip to content

Unclutter epi_archive's print method + add preview of DT #341

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

Closed
brookslogan opened this issue Jun 30, 2023 · 3 comments · Fixed by #412
Closed

Unclutter epi_archive's print method + add preview of DT #341

brookslogan opened this issue Jun 30, 2023 · 3 comments · Fixed by #412
Assignees
Labels
P1 medium priority REPL Improved print, errors, etc.

Comments

@brookslogan
Copy link
Contributor

Pretty commonly, we'll print an epi_archive x, see a lot but not very useful, then print x$DT to get a better idea what things look like. We should rebalance the print output to include the DT, exclude any redundant information, and move some other information into a summary method. Then we might be able to make DT a private field without much issue, and avoid some worries about aliasing and mutation and key invalidation.

@brookslogan brookslogan added P1 medium priority REPL Improved print, errors, etc. labels Jun 30, 2023
@brookslogan
Copy link
Contributor Author

Another note from @dsweber2 : we should consider head and tail methods. Probably after #340.

@brookslogan
Copy link
Contributor Author

Then we might be able to make DT a private field without much issue, and avoid some worries about aliasing and mutation and key invalidation.

Making DT a private field is out of scope. This issue is a blocker to it, though (as is implementing filter for epi_archives).

@brookslogan
Copy link
Contributor Author

Another note from @dsweber2 : we should consider head and tail methods. Probably after #340.

This is out of scope.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P1 medium priority REPL Improved print, errors, etc.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants