Skip to content

[DO NOT MERGE] Laghos comm dynamic #604

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

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

Conversation

gracenansamba
Copy link
Collaborator

@gracenansamba gracenansamba commented Feb 11, 2025

When git="https://github.com/gracenansamba/mfem.git" branch comm_cali is merged into MFEM:

  • remove the following line from benchpark/repo/mfem/package.py::

      version("4.4_comm_cali", branch="comm_cali", submodules=False, git="https://github.com/gracenansamba/mfem.git")
    
  • remove the following line from benchpark/repo/laghos/package.py::

    depends_on("[email protected]_comm_cali+caliper", when="+caliper") 
    

This is the experiment init I used: benchpark experiment init --dest=laghos-benchmark-dane laghos +strong ~single_node caliper=mpi

experiment.py does not looks similar to how the problems in kripke are defined

variant(
"workload",
default="triplept",
description="triplept or other problem",
)

@gracenansamba gracenansamba self-assigned this Feb 11, 2025
@github-actions github-actions bot added experiment New or modified experiment application labels Feb 11, 2025
@pearce8 pearce8 mentioned this pull request Feb 19, 2025
@pearce8
Copy link
Collaborator

pearce8 commented Feb 19, 2025

@dyokelson @gracenansamba I removed comm_cali pointers from develop, but not from this branch/PR. Hopefully I did not break your workflow....

@rfhaque rfhaque changed the title Laghos comm dynamic [[DO NOT MERGE]] Laghos comm dynamic Apr 4, 2025
@rfhaque rfhaque changed the title [[DO NOT MERGE]] Laghos comm dynamic [DO NOT MERGE] Laghos comm dynamic Apr 4, 2025
@github-actions github-actions bot added the feature New feature or request label Apr 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
application experiment New or modified experiment feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants