Skip to content

Add a timer plugin#652

Open
Yushan-Wang wants to merge 71 commits into
pdidev:mainfrom
Yushan-Wang:add-a-timer-report
Open

Add a timer plugin#652
Yushan-Wang wants to merge 71 commits into
pdidev:mainfrom
Yushan-Wang:add-a-timer-report

Conversation

@Yushan-Wang
Copy link
Copy Markdown
Member

@Yushan-Wang Yushan-Wang commented Mar 12, 2026

List of things to check before making a PR

Before merging your code, please check the following:

  • you have added a line describing your changes to the Changelog;
  • you have added unit tests for any new or improved feature;
  • in case you updated dependencies, you have checked pdi/docs/CheckList.md;
  • in case of a change in pdi.h, this same change must be reflected in no-pdi/include/pdi.h;
  • you have checked your code format:
    • you have checked that you respect all conventions specified in CONTRIBUTING.md;
    • you have checked that the indentation and formatting conforms to the .clang-format;
    • you have documented with doxygen any new or changed function / class;
  • you have correctly updated the copyright headers:
    • your institution is in the copyright header of every file you (substantially) modified;
    • you have checked that the end-year of the copyright there is the current one;
  • you have updated the AUTHORS file:
    • you have added yourself to the AUTHORS file;
    • if this is a new contribution, you have added it to the AUTHORS file;
  • you have added everything to the user documentation:
    • any new CMake configuration option;
    • any change in the yaml config;
    • any change to the public or plugin API;
    • any other new or changed user-facing feature;
    • any change to the dependencies;
  • you have correctly linked your MR to one or more issues:
    • your MR solves an identified issue;
    • your commit contain the Fix #issue keyword to autoclose the issue when merged.

@Yushan-Wang Yushan-Wang marked this pull request as ready for review May 11, 2026 12:18
@Yushan-Wang Yushan-Wang changed the title Add a timer report Add a timer plugin May 12, 2026
Comment thread CMakeLists.txt Outdated
Comment thread plugins/timer/README.md Outdated
Comment thread plugins/timer/README.md Outdated
Comment thread plugins/timer/timer.cxx Outdated
Comment thread plugins/timer/timer.cxx Outdated
Comment thread plugins/timer/timer.cxx Outdated
Comment thread example/timer.yml Outdated
Comment thread plugins/timer/tests/timer_test.cxx Outdated
Comment thread plugins/timer/tests/timer_test.cxx Outdated
Comment thread plugins/timer/tests/CMakeLists.txt Outdated
Yushan-Wang and others added 14 commits May 12, 2026 13:06
Removed the 'timer' flag from the configuration.
Co-authored-by: JAuriac <56091659+JAuriac@users.noreply.github.com>
- Introduced option to build Timer plugin in CMakeLists.txt
- Updated timer.yml to rename user timer and remove unused JSON configuration
- Adjusted timer_test properties in CMakeLists.txt for proper execution
- Cleaned up timer_test.cxx by removing unnecessary timer definitions
Updated copyright year for CEA in the file header.
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.

3 participants