Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

YAML output fix #6

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

omaralvarez
Copy link

As is, when using a YAML parser there is a Scanner error: mapping values are not allowed in this context.

This PR fixes #5.

The problematic chunk of the output is for example:

Performance Results For Rank 0:
  Timer: total
    CallCount:  1
    AvgPerCall:   6.0009
    Total:        6.0009
    PercentLoop:   101.79
  Timer: loop
    CallCount:  1
    AvgPerCall:   6.0009
    Total:        6.0009
    PercentLoop:   101.79

It should instead output:

Performance Results For Rank 0:
  - Timer: total
    CallCount:  1
    AvgPerCall:   6.0009
    Total:        6.0009
    PercentLoop:   101.79
  - Timer: loop
    CallCount:  1
    AvgPerCall:   6.0009
    Total:        6.0009
    PercentLoop:   101.79

With these changes the YAML output is compliant with the standard version 1.2.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

YAML output not compliant with standard
1 participant