Skip to content

Valid characters in algorithm names (and identifiers in general) #343

Open

Description

@beojan

Currently, algorithm_name construction from a spec checks that all characters are regex \w word characters. This doesn't seem to be checked anywhere else.

Should there be a rule (probably) and what should it be?

Activity

  1. knoepfel commented on Mar 10, 2026

    @knoepfel
    Member

    Agreement: Phlex will enforce the restriction that the following identifiers used in specifying a data product must be representable as a valid C++ variable name:

    • Plugin name (the key in the configuration document that corresponds to the loadable library)
    • Algorithm name (the string specified in the registration stanza)
    • Suffix (the user can specify the empty string)
    • Stage name
    • Data layer name element (e.g., the data-layer path /run/subrun/spill contains the following elements: run, subrun, and spill)
  2. added this to the Prototype 0.5 milestone on Sep 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    product specificationHow should data products be specified as inputs to algorithms

    Type

    No type

    Projects

    No projects

      Milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions