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.

Sign up for GitHub

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

Jump to bottom

Eqsim #213

Open
wants to merge 31 commits into
base: develop
Choose a base branch
from
Open

Eqsim #213

wants to merge 31 commits into from

Conversation

houjun
Copy link
Member

@houjun houjun commented Oct 23, 2024 •
edited
Loading

Related Issues / Pull Requests

List all related issues and/or pull requests if there are any.

Description

  • Add EQSIM-related benchmark code
  • Add (partial) support for ZFP compression
  • Fix an issue with data querying
  • Fix an issue with >1GB reads
  • Fix an issue with 2d/3d reads

What changes are proposed in this pull request?

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality not to work as expected; for instance, examples in this repository must be updated too)
  • This change requires a documentation update

Checklist:

  • My code modifies existing public API, or introduces new public API, and I updated or wrote docstrings
  • I have commented my code
  • My code requires documentation updates, and I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

houjun and others added 25 commits February 23, 2024 14:51
@houjun
Update getting_started.rst (#184)
f8d5f8d
@hyunjuooh @jeanbez
Removing gres option for ctest (#182)
38d5f77 
* Removing gres option for ctest
* Removing gres option from scripts
* Update check for core

---------

Co-authored-by: Hyunju Oh <hjoh16@login15.chn.perlmutter.nersc.gov>
Co-authored-by: Jean Luca Bez <jlbez@lbl.gov>
@jeanbez
enable cache by default (#187)
0f326da
@houjun
Benchmark code for EQSIM data
cc8c2a0
@github-actions
Committing clang-format changes
ab4d5b6
@houjun
Minor adjustments
512e714
@github-actions
Committing clang-format changes
8a5835f
@houjun
Updates
1f661bc
@github-actions
Committing clang-format changes
a7903e1
@houjun
Change vpicio to use local server partitioning, add some debug prints
b0f1c8f
@github-actions
Committing clang-format changes
788dd2e
@houjun
Add metadata query to benchmark code
dcc56a1
@github-actions
Committing clang-format changes
cf3f7a5
@houjun
Add ZFP compression for read and write
8d44125
@github-actions
Committing clang-format changes
b8840e0
@houjun
Add a option to use more ranks to read data so total data of each ran…
3bcdf53 
…k is less than the 4GB chunk limit
@github-actions
Committing clang-format changes
3e97d5d
@houjun
Add a data query code for EQSIM data
0fe3f6f
@github-actions
Committing clang-format changes
a502ac9
@houjun
Minor adjustments for the HDF5 read code
b72e981
@github-actions
Committing clang-format changes
9a3ba68
@houjun
Fix an issue with periodic data flush, minor changes to benchmark code
7a069b3
@github-actions
Committing clang-format changes
b3f618f
@houjun
fix an issue with 3d read segfault
4c9d2bb
@github-actions
Committing clang-format changes
9068c09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeanbez jeanbez Awaiting requested review from jeanbez

@zhangwei217245 zhangwei217245 Awaiting requested review from zhangwei217245

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@houjun @hyunjuooh @jeanbez