Skip to content

[Zeus] add testing in CI#711

Merged
tdavidcl merged 5 commits intomainfrom
ci/zeus-test
Dec 22, 2024
Merged

[Zeus] add testing in CI#711
tdavidcl merged 5 commits intomainfrom
ci/zeus-test

Conversation

@tdavidcl
Copy link
Copy Markdown
Member

@tdavidcl tdavidcl commented Dec 21, 2024

Add zeus to ci tests for the sod tube setup.

@tdavidcl tdavidcl linked an issue Dec 22, 2024 that may be closed by this pull request
@github-actions
Copy link
Copy Markdown
Contributor

Pre-commit check: ✅

trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check for merge conflicts................................................Passed
check that executables have shebangs.....................................Passed
check that scripts with shebangs are executable..........................Passed
check for added large files..............................................Passed
check for case conflicts.................................................Passed
check yaml...............................................................Passed
detect private key.......................................................Passed
Validate GitHub Workflows................................................Passed
clang-format.............................................................Passed
Check doxygen headers....................................................Passed
Check license headers....................................................Passed
Check #pragma once.......................................................Passed
Check SYCL #include......................................................Passed

Test pipeline can run.

@github-actions
Copy link
Copy Markdown
Contributor

Cland tidy diff report


3 warnings generated.
Suppressed 3 warnings (3 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.

4 warnings generated.
Suppressed 4 warnings (4 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.

4 warnings generated.
Suppressed 4 warnings (4 in non-user code).
Use -header-filter=.* to display errors from all non-system headers. Use -system-headers to display errors from system headers as well.

Suggested changes

Detailed changes :

@github-actions
Copy link
Copy Markdown
Contributor

Doxygen diff with main

Removed warnings : 3
New warnings : 25
Warnings count : 5714 → 5736 (0.4%)

Detailed changes :
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:25: warning: Compound shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:27: warning: Member Tscal (typedef) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:28: warning: Member Tgridscal (typedef) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:29: warning: Member dim (variable) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:31: warning: Member Config (typedef) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:32: warning: Member Storage (typedef) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:33: warning: Member AMRBlock (typedef) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:35: warning: Member context (variable) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:36: warning: Member solver_config (variable) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:37: warning: Member storage (variable) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:39: warning: Member solution (variable) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:40: warning: Member direction (variable) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:41: warning: Member time_val (variable) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:42: warning: Member x_ref (variable) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:43: warning: Member x_max (variable) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:43: warning: Member x_min (variable) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:45: warning: Member AnalysisSodTube(ShamrockCtx &context, Config &solver_config, Storage &storage, shamphys::SodTube &solution, Tvec direction, Tscal time_val, Tscal x_ref, Tscal x_min, Tscal x_max) (function) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:58: warning: Compound shammodels::zeus::modules::AnalysisSodTube::field_val is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:59: warning: Member rho (variable) of struct shammodels::zeus::modules::AnalysisSodTube::field_val is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:60: warning: Member v (variable) of struct shammodels::zeus::modules::AnalysisSodTube::field_val is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:61: warning: Member P (variable) of struct shammodels::zeus::modules::AnalysisSodTube::field_val is not documented.
+ src/shammodels/amr/zeus/modules/AnalysisSodTube.hpp:64: warning: Member compute_L2_dist() (function) of class shammodels::zeus::modules::AnalysisSodTube is not documented.
+ src/shammodels/bindings/pyAMRZeusModel.cpp:108: warning: Member Register_pymod(pyamrzeusmodel) (function) of file pyAMRZeusModel.cpp is not documented.
- src/shammodels/bindings/pyAMRZeusModel.cpp:24: warning: Member add_instance(py::module &m, std::string name_config, std::string name_model) (function) of namespace shammodels::zeus is not documented.
- src/shammodels/bindings/pyAMRZeusModel.cpp:24: warning: Member add_instance(py::module &m, std::string name_config, std::string name_model) (function) of namespace shammodels::zeus is not documented.
+ src/shammodels/bindings/pyAMRZeusModel.cpp:25: warning: Member add_instance(py::module &m, std::string name_config, std::string name_model) (function) of namespace shammodels::zeus is not documented.
+ src/shammodels/bindings/pyAMRZeusModel.cpp:25: warning: Member add_instance(py::module &m, std::string name_config, std::string name_model) (function) of namespace shammodels::zeus is not documented.
- src/shammodels/bindings/pyAMRZeusModel.cpp:80: warning: Member Register_pymod(pyamrzeusmodel) (function) of file pyAMRZeusModel.cpp is not documented.

@tdavidcl tdavidcl merged commit 14c0577 into main Dec 22, 2024
@tdavidcl tdavidcl deleted the ci/zeus-test branch December 22, 2024 21:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] add Zeus sod test in phys tests

1 participant