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

JSON parsing #29

Closed
wants to merge 18 commits into from
Closed

JSON parsing #29

wants to merge 18 commits into from

Conversation

wrightaprilm
Copy link

Hi all,

Some undergrads in my lab (@sujanamht, @Pranup19, @Anjalithapaa, @maheessh) have added functionality for parsing JSON files to the readTrace() function. This includes expanding basic parsing, as well as expanding entries that are vectors into multiple columns.

@cmt2
Copy link
Collaborator

cmt2 commented Jul 3, 2024

Hi April, this is great! Thanks for adding this. Could you add this as a feature development branch (fork from development, add the new functionality, ensure it passes checks, etc.)? Then once it's passing all checks we can merge into development. We maintain master as matching the current version on CRAN, and development as the the newest (but tested) functionality. All other more experimental functions are worked through on feature branches before making it into development, and once enough changes have accumulated, we can publish an update on CRAN.

@cmt2 cmt2 closed this Jul 3, 2024
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.

4 participants