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

Address an issue with ElasticityComponentIntegrator #207

Merged
merged 2 commits into from
Jan 29, 2024
Merged

Conversation

sshiraiwa
Copy link
Member

This small PR fix generate_bilinearinteg_ext.py to handle ElasticityComponentIntegrator.

(note) generate_bilinearinteg_ext.py is used to generate appropriate SWIG interface file for mfem::BilinearFormIntegrator so that a coefficient passed as an argument is not going
to be GCed by Python. This generator needs to be adjusted when mfem::BilinearFormIntegrator
constructor argument changes.

@sshiraiwa sshiraiwa self-assigned this Jan 29, 2024
@sshiraiwa sshiraiwa added bug in-test-with-mfem-master trigger a workflow to test using MFEM master labels Jan 29, 2024
…atch with cuda using nvidia-cuda docker image in doc/development_memo.txt
@sshiraiwa sshiraiwa added the in-test-with-mfem-release trigger a workflow to test using MFEM release label Jan 29, 2024
@sshiraiwa sshiraiwa merged commit 135f369 into master Jan 29, 2024
24 checks passed
* MFEM
git clone https://github.com/mfem/mfem.git
cd mfem
make config MFEM_USE_MPI=YES MFEM_USE_METIS_5=YES PREFIX=/usr/local SHARED=YES
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious: shouldn't CUDA be enabled in this build too? E.g. by adding MFEM_USE_CUDA=YES CUDA_ARCH=sm_70?

@sshiraiwa
Copy link
Member Author

sshiraiwa commented Jan 30, 2024 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug in-test-with-mfem-master trigger a workflow to test using MFEM master in-test-with-mfem-release trigger a workflow to test using MFEM release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants