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.

Sign up for GitHub

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

Jump to bottom

Adding a Forth Based Avro Reader #1491

Merged
merged 85 commits into from
Jun 10, 2022
Merged

Adding a Forth Based Avro Reader #1491

merged 85 commits into from
Jun 10, 2022

Conversation

aryan26roy
Copy link
Collaborator

No description provided.

aryan26roy and others added 30 commits March 7, 2022 19:17
@aryan26roy
added jupyter notebook and the prelimnary function
f10c7ed
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
6f013d7 
for more information, see https://pre-commit.ci
@aryan26roy
Added proper files
a491f43
@aryan26roy
Merge branch 'aryan-avro' of github.com:scikit-hep/awkward-1.0 into a…
ff69f4f 
…ryan-avro
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
61f8116 
for more information, see https://pre-commit.ci
@aryan26roy
Added tests skeleton
1471a62
@aryan26roy
Merge branch 'aryan-avro' of github.com:scikit-hep/awkward-1.0 into a…
67fa941 
…ryan-avro
@aryan26roy
Added f strings
94efa64
@aryan26roy
Added first test
85b7878
@aryan26roy
Cleaned up string handling
f752fe5
@aryan26roy
Fixed API along with first test.
2fc0ec2
@aryan26roy
Added more tests
dd3d660
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
3c810bd 
for more information, see https://pre-commit.ci
@aryan26roy
Added more tests (not working yet)
40607d0
@aryan26roy
Merge branch 'aryan-avro' of github.com:scikit-hep/awkward-1.0 into a…
9306298 
…ryan-avro
@aryan26roy
Added multiple sample files
0e0458b
@aryan26roy
Added null and array_enum sample files
fe0e30d
@aryan26roy
Added all sample files except unions_record_null
3b4ec83
@aryan26roy
added multiple tests
2e8c0fd
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
53211f2 
for more information, see https://pre-commit.ci
@aryan26roy
Linting
11c1c5c
@aryan26roy
Merge branch 'aryan-avro' of github.com:scikit-hep/awkward-1.0 into a…
d9575ed 
…ryan-avro
@aryan26roy
added all samples
df53b81
@aryan26roy
enums works
a2b2ce9
@aryan26roy
Records work now
f376615
@aryan26roy
Added tests for union and identified some bugs
294d4c5
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
07c4afc 
for more information, see https://pre-commit.ci
@aryan26roy
Partially fixed enum
3995ed3
@aryan26roy
Merge branch 'aryan-avro' of github.com:scikit-hep/awkward-1.0 into a…
3fc1826 
…ryan-avro
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
2914159 
for more information, see https://pre-commit.ci
@codecov
Copy link

codecov bot commented Jun 8, 2022 •
edited
Loading

Codecov Report

Merging #1491 (f504c51) into main (8a4ea2b) will increase coverage by 0.18%.
The diff coverage is 85.24%.

Impacted Files Coverage Δ
src/awkward/_v2/contents/indexedarray.py 73.83% <ø> (ø)
src/awkward/_v2/operations/ak_cartesian.py 89.68% <ø> (ø)
src/awkward/_v2/operations/ak_full_like.py 100.00% <ø> (ø)
src/awkward/_v2/operations/ak_nan_to_none.py 26.66% <ø> (ø)
src/awkward/_v2/operations/ak_with_field.py 97.87% <ø> (ø)
src/awkward/_v2/_connect/jax/__init__.py 88.88% <66.66%> (-1.59%) ⬇️
src/awkward/_v2/contents/unmaskedarray.py 68.94% <66.66%> (ø)
src/awkward/_v2/operations/ak_from_avro_file.py 66.66% <66.66%> (ø)
src/awkward/_v2/_connect/jax/_reducers.py 76.92% <76.92%> (ø)
src/awkward/_v2/contents/listoffsetarray.py 81.85% <77.77%> (ø)
... and 25 more

jpivarski and others added 18 commits June 9, 2022 09:44
@jpivarski
Fix test.
a04ecb4
@jpivarski
A little clean-up.
2f8273d
@jpivarski
Dropped the reader that makes Python code.
c191d01
@jpivarski
Tiny correction.
3f0f40b
@aryan26roy
fixed failing test
bc18b7a
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
57ec662 
for more information, see https://pre-commit.ci
@aryan26roy
Final polishing
2a2c0c0
@aryan26roy
Merge branch 'aryan-forth-reader' of github.com:scikit-hep/awkward in…
f09a1f7 
…to aryan-forth-reader
@aryan26roy
changed interface and added debug statements for CI
8095d8b
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
4fac3ea 
for more information, see https://pre-commit.ci
@aryan26roy
more debugging
899b5bb
@aryan26roy
Merge branch 'aryan-forth-reader' of github.com:scikit-hep/awkward in…
80265d0 
…to aryan-forth-reader
@pre-commit-ci
[pre-commit.ci] auto fixes from pre-commit.com hooks
82288fd 
for more information, see https://pre-commit.ci
@aryan26roy
changed paths for tests
d5aae68
@aryan26roy
changed paths
59df196
@aryan26roy
fixed path concatenation
daa01c4
@aryan26roy
fixed windows paths
924b9a9
@aryan26roy
fixed windows paths, again
f504c51
@aryan26roy aryan26roy merged commit 5f72d82 into main Jun 10, 2022
@aryan26roy aryan26roy deleted the aryan-forth-reader branch June 10, 2022 14:09
@jpivarski jpivarski mentioned this pull request Jun 30, 2022
Closed
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jpivarski jpivarski jpivarski requested changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aryan26roy @jpivarski