ron litzenberger comments

Results 12 comments of


                                            ron litzenberger

Js -> Go test fails

I have removed the JS->go tests from the runner until this is resolved. To create error run: ` node extract-js2.go.js`

Verify warmup is working as expected

With the recent changes, warmup may not be working as expected. Need to verify.

Verify warmup is working as expected

removed this feature for now.

Verify warmup is working as expected

It wasn't working and figured it was a lower priority over adding tests.

Verify warmup is working as expected

ok yep sounds good.

go cat'ing from js is significantly slower the other combinations + heap out of memory

Yes, will write this test now.

go cat'ing from js is significantly slower the other combinations + heap out of memory

Added js -> go and go -> js. Here are the results running locally: js -> go ``` ────────────────────┬────────────────────┬──────────────────────────────────────────────────┬────────────────────┬─────────────────────┐ │ Test │ Warmup │ Description │ File Set │ Duration...

go cat'ing from js is significantly slower the other combinations + heap out of memory

Clinic reports for js -> go https://upload.clinicjs.org/public/238842ad20f8542b91743e377f21ab9f455dcddfea3bc0702830c401da302bbf/34523.clinic-doctor.html https://upload.clinicjs.org/public/d0491b68df5b070fa226600ac4ce0389d04f35ab4e7f2aa0c718d5ae615101e4/34583.clinic-flame.html https://upload.clinicjs.org/public/ad48c42cc98c35249690f2a8bb8b7a05880de6e8b3f781a5c720c42e06d0489c/34730.clinic-bubbleprof.html

Architecture to add libp2p tests

@Elexy I think the js-ipfs tests in the repo are fine, he just wants to pull another repo that will need to be synced by the runner. @vasco-santos did I...

Add more Go benchmarks

Added a few go benchmarks but had to remove them due to https://github.com/ipfs/benchmarks/issues/241