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

ParmParse::prettyPrintTable #4101

Merged
merged 1 commit into from
Aug 21, 2024

Conversation

WeiqunZhang
Copy link
Member

@WeiqunZhang WeiqunZhang commented Aug 21, 2024

Add a new function that prints the ParmParse table without duplicates. Note that these are not duplicates, because they have different values.

foo.bar = 1
foo.bar = 2

These are also not considered duplicates, even though the final results are the same.

c = 3
foo.bar = c
foo.bar = 3

This should help WarpX to reduce duplicates when printing ParmParse parameters. However, there will still be "duplicates" like below because they are different strings.

geometry.prob_hi = 20.e-6 20.e-6
geometry.prob_hi = 2.0000000000000002e-05 2.0000000000000002e-05

Related to #4087

Add a new function that prints the ParmParse table without duplicates. Note
that these are not duplicates, because they have different values.

    foo.bar = 1
    foo.bar = 2

These are also not considered duplicates, even though the final results are
the same.

    c = 3
    foo.bar = c
    foo.bar = 3

This should help WarpX to reduce duplicates when printing ParmParse
parameters. However, there will still be "duplicates" like below because
they are different strings.

    geometry.prob_hi = 20.e-6 20.e-6
    geometry.prob_hi = 2.0000000000000002e-05 2.0000000000000002e-05
@WeiqunZhang WeiqunZhang requested review from ax3l and atmyers August 21, 2024 19:53
@WeiqunZhang WeiqunZhang merged commit b05f69f into AMReX-Codes:development Aug 21, 2024
71 of 72 checks passed
@WeiqunZhang WeiqunZhang deleted the pp_rm_dup branch August 21, 2024 20:51
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants