Catalyst merge requestshttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests2024-03-25T17:33:50-04:00https://gitlab.kitware.com/paraview/catalyst/-/merge_requests/110Release notes for 2.0 release2024-03-25T17:33:50-04:00Corey Wetterer-Nelson (Kitware)Release notes for 2.0 releaseAdd release notes for the 2.0 releaseAdd release notes for the 2.0 releasehttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/109gitlab-ci: run Windows jobs on ParaView runners2024-02-28T10:53:18-05:00Ben Boeckelgitlab-ci: run Windows jobs on ParaView runnershttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/108Print the searched paths on linux as on window2024-02-26T12:00:21-05:00Charles GueunetPrint the searched paths on linux as on windowAdd a missing `catalyst_debug` for linux versionAdd a missing `catalyst_debug` for linux versionhttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/107gitlab-ci: require x86_64-v3 for oneapi jobs2024-02-23T08:40:38-05:00Ben Boeckelgitlab-ci: require x86_64-v3 for oneapi jobsIt seems that v2 is not suitable for running the compiler.
---
See: https://gitlab.kitware.com/paraview/catalyst/-/jobs/9364351#L225 on `hera` (a v2 processor).
Cc: @brad.kingIt seems that v2 is not suitable for running the compiler.
---
See: https://gitlab.kitware.com/paraview/catalyst/-/jobs/9364351#L225 on `hera` (a v2 processor).
Cc: @brad.kinghttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/106Docs add citation2024-02-13T10:47:32-05:00Christos TsolakisDocs add citationAdd citation information in catalyst documentation.Add citation information in catalyst documentation.Christos TsolakisChristos Tsolakishttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/105Revert "cmake: Link to Python publicly when compiled with Python"2024-01-30T14:22:59-05:00Christos TsolakisRevert "cmake: Link to Python publicly when compiled with Python"Compiling with python does not mean that every application linking to catalyst
should require python.
See discussion in paraview/catalyst#33Compiling with python does not mean that every application linking to catalyst
should require python.
See discussion in paraview/catalyst#33Christos TsolakisChristos Tsolakishttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/104CI: Add CDashy annotations to job sidebar2024-01-25T21:32:23-05:00Ryan Krattigerryan.krattiger@kitware.comCI: Add CDashy annotations to job sidebarAdd helpful links to build-test jobAdd helpful links to build-test jobhttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/103guard for multiple definitions of PyObject2024-01-16T04:22:53-05:00Christos Tsolakisguard for multiple definitions of PyObjectPreviously, including `Python.h` and thus bringing the definition of `PyObject` but
not compiling catalyst with Python would create a typedef redefinition error
since we used to define an alternative typedef for `PyObject`.
The new impl...Previously, including `Python.h` and thus bringing the definition of `PyObject` but
not compiling catalyst with Python would create a typedef redefinition error
since we used to define an alternative typedef for `PyObject`.
The new implementation looks for the public symbol `PyObject_HEAD` which
signifies that we already included `Python.h`Christos TsolakisChristos Tsolakishttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/102ci: simplify the cdash-groups.json file2023-12-18T21:39:10-05:00Ben Boeckelci: simplify the cdash-groups.json fileWith each entry on its own line, the differences are easier to see.With each entry on its own line, the differences are easier to see.https://gitlab.kitware.com/paraview/catalyst/-/merge_requests/101impl-stub: add `$ORIGIN/..` to the install rpath2023-12-17T22:41:26-05:00Ben Boeckelimpl-stub: add `$ORIGIN/..` to the install rpathThis ensures the stub can load the main catalyst library.This ensures the stub can load the main catalyst library.https://gitlab.kitware.com/paraview/catalyst/-/merge_requests/100ci: Add jobs for classic and oneAPI Intel compilers2023-12-18T16:17:44-05:00Christos Tsolakisci: Add jobs for classic and oneAPI Intel compilersThe following 8 jobs are added
intel {"classic","oneAPI"} compiler with {internal,external} conduit {with,without} MPI.
Fortran & Python wrappings are enabled for all of these jobs
Resolves: paraview/catalyst#30The following 8 jobs are added
intel {"classic","oneAPI"} compiler with {internal,external} conduit {with,without} MPI.
Fortran & Python wrappings are enabled for all of these jobs
Resolves: paraview/catalyst#30Christos TsolakisChristos Tsolakishttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/99Expose function for converting C conduit pointer to Python object2023-12-22T10:44:31-05:00Christos TsolakisExpose function for converting C conduit pointer to Python objectWe already use `PyConduit_Node_Python_Wrap` internally for creating a `PyObject` out of a `conduit_node*`. This MR exposes this function to the consumers of catalyst.
This is required for paraview/paraview!6614.
I don't have a test h...We already use `PyConduit_Node_Python_Wrap` internally for creating a `PyObject` out of a `conduit_node*`. This MR exposes this function to the consumers of catalyst.
This is required for paraview/paraview!6614.
I don't have a test here yet but we have verified that it works as indented under linux and macos in the context of paraview/paraview!6614.
I will be adding a test as a separate MR.Christos TsolakisChristos Tsolakishttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/98add stdint.h to fix error: ‘std::uint64_t’ has not been declared2023-11-21T17:48:05-05:00Kevin Teeadd stdint.h to fix error: ‘std::uint64_t’ has not been declaredI failed to compile commit afa7ec7989b0ac8d26d0dc3fc39af11058c90d87 with this error.
cmake version 3.27.7
gcc (GCC) 13.2.0
g++ (GCC) 13.2.0
```
Operating System: openSUSE Tumbleweed 20231114
KDE Plasma Version: 5.27.9
KDE Frameworks Ve...I failed to compile commit afa7ec7989b0ac8d26d0dc3fc39af11058c90d87 with this error.
cmake version 3.27.7
gcc (GCC) 13.2.0
g++ (GCC) 13.2.0
```
Operating System: openSUSE Tumbleweed 20231114
KDE Plasma Version: 5.27.9
KDE Frameworks Version: 5.112.0
Qt Version: 5.15.11
Kernel Version: 6.6.1-1-default (64-bit)
Graphics Platform: X11
Processors: 12 × AMD Ryzen 5 4600H with Radeon Graphics
Memory: 7.6 GiB of RAM
Graphics Processor: NVIDIA GeForce GTX 1650 Ti/PCIe/SSE2
Manufacturer: LENOVO
Product Name: 82B5
System Version: Lenovo Legion 5 15ARH05
```
```
-- Build files have been written to: /run/build/Catalyst/_flatpak_build
[10/53] Building CXX object src/tools/replay/CMakeFiles/catalyst_replay.dir/catalyst_replay.cpp.o
FAILED: src/tools/replay/CMakeFiles/catalyst_replay.dir/catalyst_replay.cpp.o
/usr/bin/c++ -DCATALYST_USE_MPI -DMPICH_SKIP_MPICXX -DMPI_NO_CPPBIND -DOMPI_SKIP_MPICXX -D_MPICC_H -I/run/build/Catalyst/thirdparty/conduit/blueprint/c -I/run/build/Catalyst/_flatpak_build/thirdparty/conduit/blueprint -I/run/build/Catalyst/_flatpak_build/thirdparty/conduit/conduit -I/run/build/Catalyst/thirdparty/conduit/conduit -I/run/build/Catalyst/thirdparty/conduit/conduit/c -I/run/build/Catalyst/src/catalyst -I/run/build/Catalyst/_flatpak_build/src/catalyst -O2 -pipe -g -Wp,-D_FORTIFY_SOURCE=2 -Wp,-D_GLIBCXX_ASSERTIONS -fexceptions -fstack-protector-strong -grecord-gcc-switches -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -fno-omit-frame-pointer -g -std=c++11 -fPIE -fvisibility=hidden -fvisibility-inlines-hidden -MD -MT src/tools/replay/CMakeFiles/catalyst_replay.dir/catalyst_replay.cpp.o -MF src/tools/replay/CMakeFiles/catalyst_replay.dir/catalyst_replay.cpp.o.d -o src/tools/replay/CMakeFiles/catalyst_replay.dir/catalyst_replay.cpp.o -c /run/build/Catalyst/src/tools/replay/catalyst_replay.cpp
/run/build/Catalyst/src/tools/replay/catalyst_replay.cpp:22:55: error: ‘std::uint64_t’ has not been declared
22 | void replace_mpi_comm(conduit_cpp::Node& params, std::uint64_t communicator, int depth = 0)
| ^~~~~~~~
/run/build/Catalyst/src/tools/replay/catalyst_replay.cpp:44:62: error: ‘std::uint64_t’ has not been declared
44 | const std::string& node_dir, int num_ranks, int rank, std::uint64_t communicator)
| ^~~~~~~~
/run/build/Catalyst/src/tools/replay/catalyst_replay.cpp: In function ‘int main(int, char**)’:
/run/build/Catalyst/src/tools/replay/catalyst_replay.cpp:340:8: error: ‘uint64_t’ is not a member of ‘std’; did you mean ‘wint_t’?
340 | std::uint64_t communicator = static_cast<std::uint64_t>(MPI_Comm_c2f(MPI_COMM_WORLD));
| ^~~~~~~~
| wint_t
/run/build/Catalyst/src/tools/replay/catalyst_replay.cpp:376:77: error: ‘communicator’ was not declared in this scope; did you mean ‘ompi_communicator_t’?
376 | replay_catalyst_initialize(catalyst_data_dump_directory, num_ranks, rank, communicator);
| ^~~~~~~~~~~~
| ompi_communicator_t
[21/53] Building CXX object thirdparty/conduit/conduit/CMakeFiles/conduit.dir/conduit_data_array.cpp.o
```https://gitlab.kitware.com/paraview/catalyst/-/merge_requests/97readthedocs: require `sphinx-rtd-theme`2023-11-16T12:36:54-05:00Ben Boeckelreadthedocs: require `sphinx-rtd-theme`---
Actually tested on RTD this time. It worked: https://readthedocs.org/projects/catalyst-in-situ/builds/22568529/---
Actually tested on RTD this time. It worked: https://readthedocs.org/projects/catalyst-in-situ/builds/22568529/https://gitlab.kitware.com/paraview/catalyst/-/merge_requests/96readthedocs: fix configuration path2023-11-15T22:17:27-05:00Ben Boeckelreadthedocs: fix configuration pathhttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/95readthedocs: remove `requirements.txt`2023-11-15T20:02:23-05:00Ben Boeckelreadthedocs: remove `requirements.txt`Catalyst doesn't have one.
---
Cc: @jcfrCatalyst doesn't have one.
---
Cc: @jcfrhttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/94readthedocs: add configuration file2023-11-15T11:14:46-05:00Ben Boeckelreadthedocs: add configuration file---
Cc: @jcfr---
Cc: @jcfrhttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/93ci: use linux-x86_64 instead of linux runner tag for release jobs2023-09-07T06:23:25-04:00Christos Tsolakisci: use linux-x86_64 instead of linux runner tag for release jobsThe last release is stuck due to the deprecated runner tag
https://gitlab.kitware.com/paraview/catalyst/-/pipelines/364849
This MR should fix it.The last release is stuck due to the deprecated runner tag
https://gitlab.kitware.com/paraview/catalyst/-/pipelines/364849
This MR should fix it.Christos TsolakisChristos Tsolakishttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/92Improve documentation2023-09-01T09:24:03-04:00Christos TsolakisImprove documentation- Fill-up TODOs
- minor formatting fixes
- Add list with catalyst implementations- Fill-up TODOs
- minor formatting fixes
- Add list with catalyst implementationsChristos TsolakisChristos Tsolakishttps://gitlab.kitware.com/paraview/catalyst/-/merge_requests/91Extend ci2023-08-09T16:40:33-04:00Christos TsolakisExtend ciWhat we (74275110b5a86e52d9011f1fbc99a7f86e697fc4) already have:
* no wrappers (all platforms)
* external conduit (Linux)
* MPI on Windows and Linux
For validating the MPI configuration python and Fortran versions of the replay tests ar...What we (74275110b5a86e52d9011f1fbc99a7f86e697fc4) already have:
* no wrappers (all platforms)
* external conduit (Linux)
* MPI on Windows and Linux
For validating the MPI configuration python and Fortran versions of the replay tests are added
Added so far:
- Test wrappers (python) on macOS
- Test wrappers (python) on Windows
- Test wrappers (python,Fortran) on Linux
- Test wrappers + MPI (python,Fortran) on Linux
- Test wrappers + MPI (python) on Windows
- Test wrappers (python,Fortran) on Linux using external conduit
- Test wrappers + MPI (python,Fortran) on Linux using external conduitChristos TsolakisChristos Tsolakis