Skip to content

Conversation

@tomeichlersmith
Copy link
Member

@tomeichlersmith tomeichlersmith commented Nov 13, 2025

  • reorg YAML configs in config dir to be more clear about different config's purposes
  • drop RcFile
  • load YAML into Parameters that are more strictly typed than YAML::Node
  • recurse into sub-files that are marked with the !file YAML tag
  • move some pftool command line arguments to configs, open door to provide more detail about what boards are connected via YAML config
  • did not pass Parameters into pflib::Target constructor (which is what I want to do) to avoid conflict with Target Jeremy is developing
  • drop loop over multiple hostnames

To Do

  • rebase onto main after a first draft of the Bittware Target is implemented and refactor to prepare for Ecal classes #255
  • ~update to pass Parameters into pflib::Target ~ not doing to avoid hiding the named (required) arguments
  • include a boost::python::dict -> Parameters conversion for pypflib not doing here, but probably can do

@tomeichlersmith tomeichlersmith marked this pull request as ready for review November 15, 2025 16:59
@tomeichlersmith tomeichlersmith merged commit 486b04d into main Nov 15, 2025
3 checks passed
@tomeichlersmith tomeichlersmith deleted the yaml-parameters-infra branch November 15, 2025 16:59
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.

2 participants