pavilion2 icon indicating copy to clipboard operation
pavilion2 copied to clipboard

No capitalized sub-variables?

Open jennfshr opened this issue 3 years ago • 1 comments

Not sure if this is the expectation, but this doesn't work.

tt-rfe1:pre-packag $ cat tests/foo.yaml 
base:
    variables:
        tester: 
            - { foo: "foo", BAR: "foo"}
        
    build:
        cmds:
            - "echo {{tester.foo}}"
            - "echo {{tester.BAR}}"
tt-rfe1:pre-packag $ pav run foo
Error making tests for series 's431':
 Error loading test configs for test set 'cmd_line': In test base from /usr/projects/hpctest/jgreen/pavilion2/examples/pre-packaged/tests/foo.yaml:
Error resolving value 'echo {{tester.BAR}}' in config at 'build.cmds.1':
Could not resolve reference 'tester.BAR': Unknown sub_var: 'BAR'
echo {{tester.BAR}}
       ^

jennfshr avatar Apr 24 '22 15:04 jennfshr

Variables names in pavilion get automatically changed to lower case, but at the main variable level and sub variable level. I think we should allow camel case, and make them case sensitive instead. Make sure this works for variables, sub-variables, result_parse keys, and result_evaluate keys.

Paul-Ferrell avatar Sep 20 '22 14:09 Paul-Ferrell

#577 fixed this issue.

smehta99 avatar Jan 05 '23 23:01 smehta99