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

[ENH] Document latest query result TSVs and JSONLD examples #253

Merged
merged 5 commits into from
Dec 19, 2024

Conversation

alyssadai
Copy link
Contributor

@alyssadai alyssadai commented Dec 17, 2024

Note: the query results section needed to be updated as part of this PR because the links to the example query result TSVs are currently broken (from neurobagel/neurobagel_examples#47)

Checklist

Please leave checkboxes empty for PR reviewers

  • PR has an interpretable title with a prefix ([ENH], [FIX], [REF], [TST], [CI], [MNT], [INF]) see our Contributing Guidelines for more info)
  • PR links to GitHub issue with mention Closes #XXXX
  • Checks pass

Copy link

netlify bot commented Dec 17, 2024

Deploy Preview for neurobagel-documentation ready!

Name Link
🔨 Latest commit 7125b93
🔍 Latest deploy log https://app.netlify.com/sites/neurobagel-documentation/deploys/67610bf81796a6000874497f
😎 Deploy Preview https://deploy-preview-253--neurobagel-documentation.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@alyssadai alyssadai changed the title Update docs [ENH] Document latest query result TSVs and JSONLD examples Dec 17, 2024
@alyssadai alyssadai marked this pull request as ready for review December 17, 2024 05:35
@surchs surchs self-requested a review December 18, 2024 21:48
Copy link
Contributor

@surchs surchs left a comment

Choose a reason for hiding this comment

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

Looks good @alyssadai. Left one question about the table you edited, otherwise

🧑‍🍳

docs/user_guide/query_tool.md Show resolved Hide resolved
Copy link
Contributor

@surchs surchs left a comment

Choose a reason for hiding this comment

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

🧑‍🍳

@alyssadai alyssadai merged commit 5be5e81 into main Dec 19, 2024
6 checks passed
@alyssadai alyssadai deleted the update-docs branch December 19, 2024 00:17
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.

Update sections on query result TSVs Add example JSONLD with derivatives metadata to examples list
2 participants