feat: add ak_to_feather and ak_from_feather functions

[zbilodea]

No description provided.

[jpivarski]

Hi!

[codecov]

Codecov Report

Merging #2683 (4dba6e2) into main (4124889) will increase coverage by 0.00%.
The diff coverage is 88.57%.

Additional details and impacted files

Files Changed	Coverage Δ
src/awkward/operations/ak_to_feather.py	82.60% <82.60%> (ø)
src/awkward/operations/__init__.py	100.00% <100.00%> (ø)
src/awkward/operations/ak_from_feather.py	100.00% <100.00%> (ø)

[jpivarski]

When you're ready (when the docstrings are done and you've done "Update branch" with no errors), do "Ready for review" and assign me as the reviewer.

[jpivarski]

Oh, I just remembered something: we want the new functions to show up in the documentation, too. They need to be added to the right sections of docs/reference/toctree.txt, and then they'll show up when you press the "View deployment" button, followed by "API reference", in the left side-bar.

[jpivarski]

Nice! Just a few twisted changes, mostly described below.

Did you see the generated documentation? In the last "View Deployment" button, the documentation for the two functions are rendered at

http://preview.awkward-array.org.s3-website.us-east-1.amazonaws.com/zbilodea/add-feather/reference/generated/ak.from_feather.html

and

http://preview.awkward-array.org.s3-website.us-east-1.amazonaws.com/zbilodea/add-feather/reference/generated/ak.to_feather.html

[jpivarski]

I think it's good for this to be called feather_version, since it would be ambiguous what the "version" is referring to otherwise.

[jpivarski]

Great, thanks! I just made a few small changes before merging because they'd be easier to apply than to describe.

I'll give you a chance to make counter-edits, but I think this is ready to squash-and-merge.

[jpivarski]

@all-contributors please add @zbilodea for code

[allcontributors]

@jpivarski

I've put up a pull request to add @zbilodea! 🎉