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

feat(measurements): add cpu measurements #188

Merged
merged 21 commits into from
Aug 29, 2023

Conversation

JonasBa
Copy link
Member

@JonasBa JonasBa commented Aug 15, 2023

Repurpose the ticker to collect CPU measurements

CleanShot 2023-08-15 at 16 09 13@2x

@JonasBa JonasBa force-pushed the jb/feat/cpu-usage branch 7 times, most recently from 4ae4b11 to 5e49e11 Compare August 16, 2023 16:13
bindings/cpu_profiler.cc Outdated Show resolved Hide resolved
bindings/cpu_profiler.cc Outdated Show resolved Hide resolved
bindings/cpu_profiler.cc Show resolved Hide resolved
bindings/cpu_profiler.cc Outdated Show resolved Hide resolved
bindings/cpu_profiler.cc Outdated Show resolved Hide resolved
bindings/cpu_profiler.cc Outdated Show resolved Hide resolved
bindings/cpu_profiler.cc Outdated Show resolved Hide resolved
@JonasBa JonasBa merged commit 5e0e785 into jb/feat/memory-usage Aug 29, 2023
13 checks passed
@JonasBa JonasBa deleted the jb/feat/cpu-usage branch August 29, 2023 16:28
JonasBa added a commit that referenced this pull request Aug 29, 2023
* feat(profiling): collect heap usage

* fix test type

* fix test type

* fix(profile): dont throw if profile was never started

* fix test type

* fix(include): include functional

* fix: reorder fields

* fix: cast size to uint64_t

* fix: remove time_ref

* ref(review): remove statics

* ref(review): add const return and init without assignment

* feat(measurements): add cpu measurements (#188)

* ref(heapticker): generalize the ticker

* feat(metrics): collect cpu usage

* ref(review): remove statics

* ref(review): remove statics

* ref(cleanup): rename param

* ref(cleanup): free

* test: handle segfault

* test str

* test str

* test str

* test str

* test removing listeners

* explicit init

* explicit init

* test(measurements): add tests

* fix(ticker): adjust count

* fix(ticker): remove segfault handler

* fix(ticker): remove segfault handler

* fix(cpp): remove compiler warning

* fix(cpp): add remove safety so we dont overflow max measurements
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.

None yet

2 participants