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

chore: remove Identifier (once known as Identities) from codebase. #1845

Merged
merged 4 commits into from
Oct 28, 2022
Merged

chore: remove Identifier (once known as Identities) from codebase. #1845

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
10bede1
chore: remove Identifier (once known as Identities) from codebase.
jpivarski Oct 28, 2022
9e9907e
docs: remove old v1 C++ references in Doxygen
agoose77 Oct 28, 2022
90fac56
Simplify the f-strings in RecordArrayType and UnionArrayType.
jpivarski Oct 28, 2022
e2008d8
Merge branch 'main' into jpivarski/remove-identifiers-aka-identities
jpivarski Oct 28, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
148 changes: 18 additions & 130 deletions docs-doxygen/index.md
View file
Open in desktop

Large diffs are not rendered by default.

1 change: 0 additions & 1 deletion docs-sphinx/prepare_docstrings.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -439,7 +439,6 @@ def dofunction(link, linelink, shortname, name, astfcn):
],
["ak.layout.ArrayBuilder"],
["ak.index.Index"],
["ak.identifier.Identifier"],
[
"ak.types.ArrayType",
"ak.types.ArrayType",
Expand Down
1 change: 0 additions & 1 deletion docs-sphinx/redirects.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
"ak.layout.ByteMaskedArray.any-ext": "reference/generated/ak.contents.ByteMaskedArray.html",
"ak.layout.Content.any-ext": "reference/generated/ak.contents.Content.html",
"ak.layout.EmptyArray.any-ext": "reference/generated/ak.contents.EmptyArray.html",
"ak.layout.Identities.any-ext": "reference/generated/ak.identifier.Identifier.html",
"ak.layout.Index.any-ext": "reference/generated/ak.index.Index.html",
"ak.layout.IndexedArray.any-ext": "reference/generated/ak.contents.IndexedArray.html",
"ak.layout.IndexedOptionArray.any-ext": "reference/generated/ak.contents.IndexedOptionArray.html",
Expand Down
5 changes: 0 additions & 5 deletions docs-sphinx/reference/toctree.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -243,11 +243,6 @@
generated/ak.index.IndexU32
generated/ak.index.IndexU8.

.. toctree::
:caption: Identities for layout nodes

generated/ak.identifier.Identifier

.. toctree::
:caption: String behaviors

Expand Down
1 change: 0 additions & 1 deletion src/awkward/__init__.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@

# layout classes; functionality that used to be in C++ (in Awkward 1.x)
import awkward.index
import awkward.identifier
import awkward.contents
import awkward.record
import awkward.types
Expand Down
2 changes: 1 addition & 1 deletion src/awkward/_broadcasting.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ def broadcast_pack(inputs, isscalar):
elif isinstance(x, Content):
nextinputs.append(
RegularArray(
x, x.length if x.nplike.known_shape else 1, 1, None, None, x.nplike
x, x.length if x.nplike.known_shape else 1, 1, None, x.nplike
)
)
isscalar.append(False)
Expand Down
Loading