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

print of teal_data should include datanames (and also possibly join_keys) #326

Open
3 tasks done
pawelru opened this issue Aug 16, 2024 · 0 comments
Open
3 tasks done

Comments

@pawelru
Copy link
Contributor

pawelru commented Aug 16, 2024

Feature description

teal might throw that data object (of teal_data class) is missing datanames attribute however this is not very straightforward to check its content. One needs to know teal.data package API to get to know this. I think we can make it simpler and print this when printing the whole object so that the knowledge of package API is no longer needed.

Currently, only validated attribute is included.

Function to modify:

setMethod("show", signature = "teal_data", function(object) {
if (object@verified) {
cat("\u2705\ufe0e", "verified teal_data object\n")
} else {
cat("\u2716", "unverified teal_data object\n")
}
rlang::env_print(object@env)
})

Code of Conduct

  • I agree to follow this project's Code of Conduct.

Contribution Guidelines

  • I agree to follow this project's Contribution Guidelines.

Security Policy

  • I agree to follow this project's Security Policy.
@pawelru pawelru changed the title print of teal_data should include datanames and also possibly join_keys print of teal_data should include datanames (and also possibly join_keys) Aug 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant