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

Publish benchmarks app #9

Closed
jsonnull opened this issue Dec 5, 2017 · 8 comments
Closed

Publish benchmarks app #9

jsonnull opened this issue Dec 5, 2017 · 8 comments

Comments

@jsonnull
Copy link

jsonnull commented Dec 5, 2017

I'd love for the app the README benchmarks are based on to be published, so that these can be analyzed and improved.

Personally, any time I see benchmarks I want to see:

  • How can I reproduce?
  • How many runs are the benchmarks based on?
  • Was best run or average run considered?
  • What were the configurations? Caching in the parcel config was mentioned, but what about caching capabilities of webpack, babel, etc...?
@devongovett
Copy link
Member

Unfortunately, I can't publish that app as it is closed source. Do you know of a reasonably large open-source app that we could try benchmarking against?

@elrumordelaluz
Copy link

Probably not huge, but why not one from here https://github.com/gothinkster/realworld

@benderTheCrime
Copy link

Happy to create a separate issue as well, but I'd also like to see comparisons against Rollup and some other bundlers!

@azu
Copy link

azu commented Dec 10, 2017

I'm interested in the source-map setting of the benchmark app(webpack and browserify).
Because, parcel-bundler does not support source-map yet #68
We have known that generating source-map is very slow.

bors bot added a commit to jser/jser.github.io that referenced this issue Dec 11, 2017
466: 2017-12-11 JS: Parcel、webpack 4.0.0α、Node.js Performance r=azu a=azu

- [📦 Parcel](https://parceljs.org/ "📦 Parcel")
- [🚀 Announcing Parcel: A blazing fast, zero configuration web application bundler 📦](https://hackernoon.com/announcing-parcel-a-blazing-fast-zero-configuration-web-application-bundler-feac43aac0f1)
- [🔌 Plugins](https://parceljs.org/plugins.html)
- Benchmark issue
  - [Publish benchmarks app · Issue #9 · parcel-bundler/parcel](parcel-bundler/parcel#9)
  - [Feature request: source map support · Issue #68 · parcel-bundler/parcel](parcel-bundler/parcel#68)
- [webpack 4.0.0-alpha.0 feedback · Issue #6064 · webpack/webpack](webpack/webpack#6064)
  - [Vote](https://webpack.js.org/vote/)
  - [Sean Thomas Larkin on Twitter: "Alright #JavaScript #webpack folks, I've created the voting item! If you _really_ want a Zero Configuration (aka your configs live in .pos… https://t.co/w5WE8T0I0J"](https://twitter.com/TheLarkInn/status/938928029044170752)
- [Node.js Performance 改善ガイド - from scratch](http://yosuke-furukawa.hatenablog.com/entry/2017/12/05/125517)
shawwn pushed a commit to shawwn/parcel that referenced this issue Jan 5, 2018
@jsonnull
Copy link
Author

jsonnull commented May 3, 2018

@DeMoorJasper why was this issue closed?

The pull request which referenced this issue swaps out the JSON parser used internally by parcel. It does not include any performance benchmarks or notes related to performance.

@DeMoorJasper
Copy link
Member

There are various examples and blog posts showing that parcel is faster than webpack.
The big project referred to in the readme is private, so thought it would be safe to close this old issue.
If you want me to reopen I will.

@jsonnull
Copy link
Author

jsonnull commented May 3, 2018

Every question in this thread aside from "can you publish the benchmark app from the readme" has not been answered.

Since the benchmarks in the README are the most visible to users of the project, it seems reasonable to use a public app to ensure that a fair comparison was made.

Now that parcel is older, in addition to the above unanswered questions I would like to know:

  • Which versions of the projects are being compared?
  • Which features of those projects are used? It should be an apples-to-apples comparison.

@mpalmr
Copy link

mpalmr commented Apr 2, 2019

This looks really shifty and although Parcel looks like good tech, this particular issue really taints it in my mind. Why not just write another benchmark? I wouldn't mind doing so myself. When I look at benchmarks like this it jus makes me think the authors of Parcel are being straight up disingenuous.

padmaia added a commit that referenced this issue Jun 5, 2020
Implement workerize transformer

Approved-by: Will Binns-Smith
gorakong pushed a commit that referenced this issue Apr 29, 2021
Merge v2 2020-12-08

Approved-by: Brian Do
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants