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

Document thoughts on specification #69

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Document thoughts on specification #69

wants to merge 1 commit into from

Conversation

mgirlich
Copy link
Owner

@krlmlr I have written up some thoughts on the tibblify() specification. It would be great if you could have a look at it whether it makes sense to you and which conclusions we can draw from this.

Key notes:

  • tib_*() specifies the input and the output format.
  • tibblify() should be strict about the input format.
  • I think tib_*() should capture the "idea" of the output format. Therefore, tib_col() should rather be an argument to tib_df().
  • Overview of formats that should be possible to parse.
  • Representation as a tibble

Key questions:

  • Are there other input formats that should be supported?
  • Does one of the tibble representations make sense?

Scope question

  • The main motivation for tibblify was to be used with JSON resp. data resulting from parsing JSON. Can it also be used outside of this context? Would this influence the API

@krlmlr
Copy link
Collaborator

krlmlr commented Jun 23, 2022

Thanks. I don't have enough mind space to do a deep dive now.

Do we have a comprehensive overview of the JSON and nested R list constructs that should be processible with tibblify? Should this be part of this documentation, could action items be deducted from that (e.g. #97)?

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

Successfully merging this pull request may close these issues.

2 participants