Growable Buffer for ForthMachine

Open aryan26roy opened this issue 2 years ago • 3 comments

Jun 24 '22 16:06 aryan26roy

Codecov Report

Merging #1517 (4081228) into main (0c6ae03) will decrease coverage by 2.66%. The diff coverage is 84.89%.

Impacted Files	Coverage Δ
...awkward/_v2/_connect/rdataframe/from_rdataframe.py	`0.00% <0.00%> (ø)`
...c/awkward/_v2/_connect/rdataframe/to_rdataframe.py	`0.00% <0.00%> (ø)`
src/awkward/_v2/types/_awkward_datashape_parser.py	`47.66% <ø> (ø)`
src/awkward/_v2/_connect/cling.py	`25.99% <9.09%> (-0.35%)`	:arrow_down:
src/awkward/_v2/types/recordtype.py	`87.06% <91.66%> (+0.83%)`	:arrow_up:
src/awkward/_v2/types/type.py	`96.46% <96.10%> (-1.27%)`	:arrow_down:
src/awkward/_v2/_lookup.py	`97.51% <100.00%> (+0.01%)`	:arrow_up:
src/awkward/_v2/types/__init__.py	`100.00% <100.00%> (ø)`
src/awkward/_v2/types/regulartype.py	`90.47% <0.00%> (+4.76%)`	:arrow_up:

Jun 24 '22 17:06 codecov[bot]

@aryan26roy - when https://github.com/scikit-hep/awkward/pull/1560 is merged you may want to rebase this PR.

Jul 27 '22 11:07 ianna

It might be easier to finish this in a new PR. The header-only GrowableBuffer is no longer changing (stable target).

Aug 24 '22 19:08 jpivarski

I'm closing this PR, with issue #1846 as a reminder that it should be picked up again (and a link back here to use this as a guide).

Oct 28 '22 20:10 jpivarski