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

Add basic benchmark suite #451

Merged
merged 1 commit into from
Aug 30, 2023
Merged

Add basic benchmark suite #451

merged 1 commit into from
Aug 30, 2023

Conversation

jsumners
Copy link
Member

@jsumners jsumners commented Aug 30, 2023

As noted in the script, we don't really care about these numbers, but they should help us understand how features affect performance.

On my M1, using 18.17.1, I get:

basicLog*10000: 776.762ms
objectLog*10000: 656.023ms
coloredLog*10000: 729.25ms
customPrettifiers*10000: 740.291ms
logWithErrorObject*10000: 564.892ms
logRemappedMsgErrKeys*10000: 580.711ms
messageFormatString*10000: 945.966ms
basicLog*10000: 727.929ms
objectLog*10000: 654.281ms
coloredLog*10000: 736.466ms
customPrettifiers*10000: 724.02ms
logWithErrorObject*10000: 556.315ms
logRemappedMsgErrKeys*10000: 558.546ms
messageFormatString*10000: 918.586ms

@jsumners jsumners requested a review from mcollina August 30, 2023 13:41
@jsumners jsumners mentioned this pull request Aug 30, 2023
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@jsumners jsumners merged commit b4a1fb6 into master Aug 30, 2023
10 checks passed
@jsumners jsumners deleted the basic-bench branch August 30, 2023 22:54
renovate bot referenced this pull request in fwouts/previewjs Oct 4, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [pino-pretty](https://togithub.com/pinojs/pino-pretty) | [`^10.2.0` ->
`^10.2.3`](https://renovatebot.com/diffs/npm/pino-pretty/10.2.0/10.2.3)
|
[![age](https://developer.mend.io/api/mc/badges/age/npm/pino-pretty/10.2.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/pino-pretty/10.2.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/pino-pretty/10.2.0/10.2.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/pino-pretty/10.2.0/10.2.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>pinojs/pino-pretty (pino-pretty)</summary>

###
[`v10.2.3`](https://togithub.com/pinojs/pino-pretty/releases/tag/v10.2.3)

[Compare
Source](https://togithub.com/pinojs/pino-pretty/compare/v10.2.2...v10.2.3)

#### What's Changed

- Manually require and export all util scripts by
[@&#8203;marvinruder](https://togithub.com/marvinruder) in
[https://github.com/pinojs/pino-pretty/pull/467](https://togithub.com/pinojs/pino-pretty/pull/467)

#### New Contributors

- [@&#8203;marvinruder](https://togithub.com/marvinruder) made their
first contribution in
[https://github.com/pinojs/pino-pretty/pull/467](https://togithub.com/pinojs/pino-pretty/pull/467)

**Full Changelog**:
pinojs/pino-pretty@v10.2.2...v10.2.3

###
[`v10.2.2`](https://togithub.com/pinojs/pino-pretty/releases/tag/v10.2.2)

[Compare
Source](https://togithub.com/pinojs/pino-pretty/compare/602be25a622812f8fbc2be318d4338a718c32902...v10.2.2)

#### What's Changed

- Add basic benchmark suite by
[@&#8203;jsumners](https://togithub.com/jsumners) in
[https://github.com/pinojs/pino-pretty/pull/451](https://togithub.com/pinojs/pino-pretty/pull/451)
- Re-org code (retry) by
[@&#8203;jsumners](https://togithub.com/jsumners) in
[https://github.com/pinojs/pino-pretty/pull/453](https://togithub.com/pinojs/pino-pretty/pull/453)
- Bump tsd from 0.28.1 to 0.29.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/pinojs/pino-pretty/pull/454](https://togithub.com/pinojs/pino-pretty/pull/454)
- Bump actions/checkout from 3 to 4 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/pinojs/pino-pretty/pull/461](https://togithub.com/pinojs/pino-pretty/pull/461)
- Disable FinalizationRegistry if NODE_V8\_COVERAGE is set by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[https://github.com/pinojs/pino-pretty/pull/464](https://togithub.com/pinojs/pino-pretty/pull/464)

**Full Changelog**:
pinojs/pino-pretty@v10.2.0...v10.2.2

###
[`v10.2.1`](https://togithub.com/pinojs/pino-pretty/compare/v10.2.0...602be25a622812f8fbc2be318d4338a718c32902)

[Compare
Source](https://togithub.com/pinojs/pino-pretty/compare/v10.2.0...602be25a622812f8fbc2be318d4338a718c32902)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - At any time (no schedule defined),
Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about these
updates again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/fwouts/previewjs).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4wLjMiLCJ1cGRhdGVkSW5WZXIiOiIzNy4wLjMiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
bodinsamuel referenced this pull request in specfy/specfy Nov 3, 2023
[![Mend
Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com)

This PR contains the following updates:

| Package | Change | Age | Adoption | Passing | Confidence |
|---|---|---|---|---|---|
| [pino-pretty](https://togithub.com/pinojs/pino-pretty) | [`10.2.0` ->
`10.2.3`](https://renovatebot.com/diffs/npm/pino-pretty/10.2.0/10.2.3) |
[![age](https://developer.mend.io/api/mc/badges/age/npm/pino-pretty/10.2.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![adoption](https://developer.mend.io/api/mc/badges/adoption/npm/pino-pretty/10.2.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![passing](https://developer.mend.io/api/mc/badges/compatibility/npm/pino-pretty/10.2.0/10.2.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|
[![confidence](https://developer.mend.io/api/mc/badges/confidence/npm/pino-pretty/10.2.0/10.2.3?slim=true)](https://docs.renovatebot.com/merge-confidence/)
|

---

### Release Notes

<details>
<summary>pinojs/pino-pretty (pino-pretty)</summary>

###
[`v10.2.3`](https://togithub.com/pinojs/pino-pretty/releases/tag/v10.2.3)

[Compare
Source](https://togithub.com/pinojs/pino-pretty/compare/v10.2.2...v10.2.3)

#### What's Changed

- Manually require and export all util scripts by
[@&#8203;marvinruder](https://togithub.com/marvinruder) in
[https://github.com/pinojs/pino-pretty/pull/467](https://togithub.com/pinojs/pino-pretty/pull/467)

#### New Contributors

- [@&#8203;marvinruder](https://togithub.com/marvinruder) made their
first contribution in
[https://github.com/pinojs/pino-pretty/pull/467](https://togithub.com/pinojs/pino-pretty/pull/467)

**Full Changelog**:
pinojs/pino-pretty@v10.2.2...v10.2.3

###
[`v10.2.2`](https://togithub.com/pinojs/pino-pretty/releases/tag/v10.2.2)

[Compare
Source](https://togithub.com/pinojs/pino-pretty/compare/602be25a622812f8fbc2be318d4338a718c32902...v10.2.2)

#### What's Changed

- Add basic benchmark suite by
[@&#8203;jsumners](https://togithub.com/jsumners) in
[https://github.com/pinojs/pino-pretty/pull/451](https://togithub.com/pinojs/pino-pretty/pull/451)
- Re-org code (retry) by
[@&#8203;jsumners](https://togithub.com/jsumners) in
[https://github.com/pinojs/pino-pretty/pull/453](https://togithub.com/pinojs/pino-pretty/pull/453)
- Bump tsd from 0.28.1 to 0.29.0 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/pinojs/pino-pretty/pull/454](https://togithub.com/pinojs/pino-pretty/pull/454)
- Bump actions/checkout from 3 to 4 by
[@&#8203;dependabot](https://togithub.com/dependabot) in
[https://github.com/pinojs/pino-pretty/pull/461](https://togithub.com/pinojs/pino-pretty/pull/461)
- Disable FinalizationRegistry if NODE_V8\_COVERAGE is set by
[@&#8203;mcollina](https://togithub.com/mcollina) in
[https://github.com/pinojs/pino-pretty/pull/464](https://togithub.com/pinojs/pino-pretty/pull/464)

**Full Changelog**:
pinojs/pino-pretty@v10.2.0...v10.2.2

###
[`v10.2.1`](https://togithub.com/pinojs/pino-pretty/compare/v10.2.0...602be25a622812f8fbc2be318d4338a718c32902)

[Compare
Source](https://togithub.com/pinojs/pino-pretty/compare/v10.2.0...602be25a622812f8fbc2be318d4338a718c32902)

</details>

---

### Configuration

📅 **Schedule**: Branch creation - "after 4pm every weekday" in timezone
Europe/Paris, Automerge - At any time (no schedule defined).

🚦 **Automerge**: Enabled.

♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the
rebase/retry checkbox.

🔕 **Ignore**: Close this PR and you won't be reminded about this update
again.

---

- [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check
this box

---

This PR has been generated by [Mend
Renovate](https://www.mend.io/free-developer-tools/renovate/). View
repository job log
[here](https://developer.mend.io/github/specfy/specfy).

<!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4zMS41IiwidXBkYXRlZEluVmVyIjoiMzcuMzEuNSIsInRhcmdldEJyYW5jaCI6ImNob3JlL3Jlbm92YXRlQmFzZUJyYW5jaCJ9-->

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
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.

2 participants