Skip to content

Pi add json docs#119

Open
pendingintent wants to merge 7 commits into
mainfrom
pi-add-json-docs
Open

Pi add json docs#119
pendingintent wants to merge 7 commits into
mainfrom
pi-add-json-docs

Conversation

@pendingintent

Copy link
Copy Markdown
Collaborator

I have reorganized the data directory.

  • Added README.md to outline new directory structure
  • Created archive directory for storiung superseded USDM JSON files
  • Moved original pilot study USDM JSON to the archive directory
  • Added latest USDM JSON for the LZZT and HCT07197120 studies. These were generated with the latest version of the SoA Workbench
  • Added latest Define JSON for the LZZT and HCT07197120 studies. These were generated with the latest version of the SoA Workbench

@pendingintent pendingintent added documentation Improvements or additions to documentation enhancement New feature or request labels Jun 18, 2026
@lexjansen

Copy link
Copy Markdown
Collaborator

Thanks! A few questions:
Will the \data\soa_workbench\protocols<protocol> folder eventually contain a collection of JSON files? If yes, would it be better than that the latest files have 'latest' in their name instead of the , so that people do not have to update their program when there is a new file.
In the README is says that the define JSON files are ODM 1.3.2 Define-XML export, but they are DDS exports. We also have to fix that in the model. It is not ODM 1.3.2.

@pendingintent

Copy link
Copy Markdown
Collaborator Author

I agree that adding latest would be best. It is my intention to archive old versions of the files. I will make these changes and re-push.

@lexjansen lexjansen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Placement of files as expected.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants