Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • bob.bio.base bob.bio.base
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 15
    • Issues 15
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • bobbob
  • bob.bio.basebob.bio.base
  • Merge requests
  • !260

Update path to key conversion to enable using paths with extensions

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Laurent COLBOIS requested to merge flexible-sample-path into master Aug 13, 2021
  • Overview 3
  • Commits 1
  • Pipelines 1
  • Changes 1

This is a very small change that would greatly facilitate writing CSVDataset interfaces for databases with multiple file extensions (cf. #160 (closed)).

I propose to simply strip the extension from the file path when creating the sample key, which enables to also use file paths containing extensions.

Nothing needs to be changed for the data loading : if using paths with extension, we can just set extension="" when initializing the CSVToSampleLoaderBiometrics.

Edited Aug 13, 2021 by Laurent COLBOIS
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: flexible-sample-path