Skip to content

Conversation

@maxfirmbach
Copy link
Contributor

@maxfirmbach maxfirmbach commented Jul 4, 2025

Description and Context

As the title states.

Related Issues and Pull Requests

Closes #968 #675 #661 #192

@maxfirmbach
Copy link
Contributor Author

I updated the wrong Trilinos hash ... before I fix that I'll just see if everything passes our test pipelines.

@maxfirmbach
Copy link
Contributor Author

Locally it works for me, it seems like in the docker image SuperLU_dist does weird things ... not sure if the versions match.

@maxfirmbach maxfirmbach force-pushed the docker-update-refactor-install-and-trilinos-update branch from 3643ad2 to c888bf2 Compare July 4, 2025 18:16
@maxfirmbach
Copy link
Contributor Author

@eulovi the following tests are failing, due to tolerances not reached:

embeddedmesh_parallel_bending_beam_fine.4C.yaml-p3 (Failed)
embeddedmesh_2d_hertzian_contact.yaml-p2-csv_comparison-xxx_energy.csv (Failed)
embeddedmesh_parallel_bending_beam_fine-vtk-structure.4C.yaml-p3 (Not Run)
embeddedmesh_block_nurbs27_hex8_wedge6-vtk.4C.yaml-p2 (Failed)

Would you mind looking at the results and check if they are acceptable? And if yes fix the relevant tests?

@maxfirmbach
Copy link
Contributor Author

Seems like this PR triggers a really nice random behavior related to contact resulting in the failing of single tests on each pipeline run, just as usual.

@mayrmt
Copy link
Member

mayrmt commented Jul 8, 2025

I am fine with the current changes.

@maxfirmbach
Copy link
Contributor Author

@ischeider @sebproell Any idea why the documentation pipeline fails?

@amgebauer
Copy link
Member

@ischeider @sebproell Any idea why the documentation pipeline fails?

we treat warnings as errors in the documentation pipeline. There is the following warning in the pipeline:

/__w/4C/4C/build/doc/tmp/installation/installation.rst:102: WARNING: Inline strong start-string without end-string. [docutils]

@maxfirmbach
Copy link
Contributor Author

@amgebauer I've just seen there's a build option to build with the documentation, will try that out locally! Thanks also for the hint about warning as errors.

@maxfirmbach maxfirmbach force-pushed the docker-update-refactor-install-and-trilinos-update branch from 9250f19 to 1f4596d Compare July 14, 2025 15:40
@maxfirmbach
Copy link
Contributor Author

Leaving this in Draft-PR mode for now, there are a few things that seem to go wrong (tests randomly failing).

@maxfirmbach
Copy link
Contributor Author

Trilinos 16.2 will be the last version with Epetra. I wait for it's release, update this and make this the only allowed version to use with 4C. I will revert some things related to the install process here again.

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.

[docker] Install more dependencies via apt install

4 participants