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

WIP: Add Dump strategy to allow user configurable IO dumps #80

Open
wants to merge 109 commits into
base: devel
Choose a base branch
from

Conversation

rfbird
Copy link
Contributor

@rfbird rfbird commented Nov 6, 2019

Still missing:

  • Make the other backends (HDF5 / OpenPMD) use DumpParams

Dave Nystrom and others added 30 commits December 30, 2018 16:39
…NL ATS-1 and CTS-1 machines.

Document how to use these two scripts.
…ment

starts out as the Cray default of PrgEnv-intel. This change checks for the
case where the user has modified their module environment and swaps it back
to the case assumed by the build script.
…le sort

implementation. Add build script support for a few more CMake variables that
were missing and should be availble to users of the build scripts.
@rfbird rfbird changed the base branch from master to devel November 6, 2019 16:51
This was referenced Dec 4, 2019
…it's not known when it's currently constructed. I instead now pass it in after user init, but the interface is a mess
…ease write chunk size to 256MB. also fix bug where pmd particles were not being flushed at proper times
@codecov
Copy link

codecov bot commented Nov 11, 2020

Codecov Report

Merging #80 (c3a63fc) into devel (170033f) will decrease coverage by 21.67%.
The diff coverage is 73.15%.

Impacted file tree graph

@@             Coverage Diff             @@
##            devel      #80       +/-   ##
===========================================
- Coverage   83.37%   61.70%   -21.68%     
===========================================
  Files         118      117        -1     
  Lines        7322    10500     +3178     
  Branches     1124     1489      +365     
===========================================
+ Hits         6105     6479      +374     
- Misses        734     3531     +2797     
- Partials      483      490        +7     
Impacted Files Coverage Δ
src/util/io/FileIO.h 100.00% <ø> (ø)
src/vpic/vpic.h 84.74% <ø> (-2.27%) ⬇️
src/grid/partition.cc 38.96% <50.00%> (+2.47%) ⬆️
src/vpic/dump.cc 64.13% <69.81%> (-5.41%) ⬇️
src/vpic/dump_strategy.cc 73.41% <73.41%> (ø)
deck/main.cc 58.33% <100.00%> (-14.01%) ⬇️
src/species_advance/species_advance.cc 67.12% <100.00%> (-12.33%) ⬇️
src/vpic/dump_strategy.h 100.00% <100.00%> (ø)
src/vpic/vpic.cc 60.31% <100.00%> (-39.69%) ⬇️
test/unit/energy_comparison/3d_test.cc 97.98% <100.00%> (-0.02%) ⬇️
... and 55 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 170033f...c3a63fc. Read the comment docs.

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.

1 participant