Skip to content
This repository has been archived by the owner on Sep 14, 2023. It is now read-only.

'100 arrays dict' fails #20

Closed
RSabet opened this issue Jul 5, 2018 · 4 comments
Closed

'100 arrays dict' fails #20

RSabet opened this issue Jul 5, 2018 · 4 comments
Labels
bug Something isn't working

Comments

@RSabet
Copy link
Collaborator

RSabet commented Jul 5, 2018

using the benchmarks from rapidjson 100 arrays dict fails with hyperjson.

@RSabet RSabet added the bug Something isn't working label Jul 7, 2018
@wdv4758h
Copy link
Contributor

wdv4758h commented Jul 9, 2018

This looks like a small case that which crash on my end.

import hyperjson                 
                                 
hyperjson.dumps([{'a': 0}]*10000)

@mre
Copy link
Owner

mre commented Nov 19, 2018

Just tried again on my machine and it works now in master, most likely due to fixes in pyo3 0.5. 🎉
@konstin fyi.

@mre mre closed this as completed Nov 19, 2018
@konstin
Copy link
Collaborator

konstin commented Nov 19, 2018

I hope this has been fixed by 0.5.0 and is not another instance of PyO3/pyo3#271, which is tackled in PyO3/pyo3#281

@mre
Copy link
Owner

mre commented Nov 19, 2018

At least the output is as expected:

>>> hyperjson.dumps([{'a': 0}]*10000)
'[{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},{"a":0},...,{"a":0},{"a":0}]'

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants