Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[compare.py] The compare tool is too verbose by default!! #153

Open
clementval opened this issue Jun 4, 2018 · 0 comments
Open

[compare.py] The compare tool is too verbose by default!! #153

clementval opened this issue Jun 4, 2018 · 0 comments
Assignees
Labels

Comments

@clementval
Copy link
Collaborator

clementval commented Jun 4, 2018

compare.py is displaying too many information by default that we do not care about.

In this example, the field data match and just one meta-data differ. But the -w option is passed then we should not have all these useless lines in the output.

[ RUN      ] mo_model_domain__p_patch(1)%cells%dist_io_data%redistrib_pattern%send_startidx
  Field meta-info mismatch: meta-info
    Expected:
      __bytesperelement: 4
      __elementtype: int
      __iminushalosize: 1
      __iplushalosize: 1
      __isize: 1
      __jminushalosize: 0
      __jplushalosize: 0
      __jsize: 0
      __kminushalosize: 0
      __kplushalosize: 0
      __ksize: 0
      __lminushalosize: 0
      __lplushalosize: 0
      __lsize: 0
      __name: mo_model_domain__p_patch(1)%cells%dist_io_data%redistrib_pattern%send_startidx
      __rank: 1
      ftg:loc: 65E6950
      ftg:registered_only: False

    Actual:
      __bytesperelement: 4
      __elementtype: int
      __iminushalosize: 1
      __iplushalosize: 1
      __isize: 1
      __jminushalosize: 0
      __jplushalosize: 0
      __jsize: 0
      __kminushalosize: 0
      __kplushalosize: 0
      __ksize: 0
      __lminushalosize: 0
      __lplushalosize: 0
      __lsize: 0
      __name: mo_model_domain__p_patch(1)%cells%dist_io_data%redistrib_pattern%send_startidx
      __rank: 1
      ftg:loc: 2459180
      ftg:registered_only: False

I propose to have a stricter option -d to disable any meta-data printing. The -w option should just show the specific meta-data that differ and not all of them.

@clementval clementval self-assigned this Jun 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant