Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • F funpack
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1
    • Issues 1
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • FSLFSL
  • funpack
  • Merge requests
  • !88

BF: Hierarchy encoding node IDs are not all sequential

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Paul McCarthy requested to merge bf/hierarchy into master May 04, 2022
  • Overview 0
  • Commits 14
  • Pipelines 4
  • Changes 22

I had mistakenly assumed that the node IDs in UKB hierarchical encodings were sequebtail, ranging from 1 to the number of nodes(e.g. the ICD10 coding). But this is not the case (e.g. coding 3).

This MR adjusts the Hierarchy class so that it no longer makes any assumptions about the node IDs.

Also included in this MR:

  • Built in field/encoding tables updated to latest from showcase
  • Ensure encoding values are loaded with the correct type - for example, coding 87 uses values comprised of digits, but which are not numeric (e.g. leading zeros).
  • New numeric/convertNumeric options to flattenHierarchical, so it can work with node IDs instead of coding labels (as the latter ar not necessarily unique for non-leaf nodes)
  • New documentation page for cleaning/processing functions
Edited May 05, 2022 by Paul McCarthy
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: bf/hierarchy