Skip to content

Commit 9447973

Browse files
anslpaPProfizi
andauthored
Fix deprecation of numpy.float and numpy.int (#734)
* Try new numpy package * Revert "Try new numpy package" This reverts commit 630d437. * Remove requirement on numpy in setup.py * Bump numpy version in requirements/requirements_install.txt * Update np.int with np.int32 * Update np.float references of dpf-core source code * Update np.float references in tests * Add --find-links to docs, examples, gate, post CI templates that it can install wheels from standalone branch * Add gate in .github folder to test it * Remove gate from .github * Update unit test in test_geometry * Update docker tests run with extra-pip-args * Add dpf_wheels_directory to CI (tests, for retro) * Add ansys-dpf-gate wheel * update dtype in docstring * to revert: test * Update test * Add check * add test * Add test * Move check step * Update dpf_wheels_directory * add check * update ci * Move check * Update docker step * test adding ROOT_SPACE * Update CI * Try source dir * remove failing check * Removing failing on docstring (different on windows and on linux) * Docker step updated * Redo Docker_test * Update test_geometry.py * Fix expected raised error type in test_geometry.py * Remove numpy pin in pyproject.toml * Fix expected raised error type in test_geometry.py * Remove test gate package since it has been released --------- Co-authored-by: PProfizi <[email protected]>
1 parent e8cc270 commit 9447973

File tree

13 files changed

+95
-92
lines changed

13 files changed

+95
-92
lines changed

.github/workflows/ci.yml

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,7 @@ jobs:
113113
ANSYS_VERSION: "231"
114114
python_versions: '["3.8"]'
115115
DOCSTRING: false
116+
custom-wheels: './.github'
116117
secrets: inherit
117118

118119
retro_222:
@@ -123,6 +124,7 @@ jobs:
123124
ANSYS_VERSION: "222"
124125
python_versions: '["3.8"]'
125126
DOCSTRING: false
127+
custom-wheels: './.github'
126128
secrets: inherit
127129

128130
retro_221:
@@ -133,6 +135,7 @@ jobs:
133135
ANSYS_VERSION: "221"
134136
python_versions: '["3.8"]'
135137
DOCSTRING: false
138+
custom-wheels: './.github'
136139
secrets: inherit
137140

138141
pydpf-post:

.github/workflows/examples.yml

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,6 @@ on:
5959
env:
6060
PACKAGE_NAME: ansys-dpf-core
6161
MODULE: core
62-
extra: "--find-links .github/"
6362

6463
jobs:
6564
setup:

.github/workflows/gate.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,6 @@ on:
3939
env:
4040
PACKAGE_NAME: ansys-dpf-core
4141
MODULE: core
42-
extra: "--find-links .github/"
4342

4443
jobs:
4544
setup:
@@ -88,7 +87,7 @@ jobs:
8887
install_extras: plotting
8988
wheelhouse: false
9089
wheel: false
91-
extra-pip-args: ${{ env.extra }}
90+
extra-pip-args: ${{ format('--find-links ./dpf-standalone/v{0}/dist', inputs.ANSYS_VERSION) }}
9291
standalone_suffix: ${{ inputs.standalone_suffix }}
9392
custom-requirements: ${{ inputs.custom-requirements }}
9493

pyproject.toml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ dependencies = [
2929
"packaging",
3030
"setuptools",
3131
"tqdm",
32-
"numpy<1.24",
32+
"numpy",
3333
"ansys-dpf-gate>=0.3.*",
3434
]
3535

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
ansys-dpf-gate==0.3.0
2-
numpy==1.23.4
2+
numpy==1.24.1
33
packaging==23.0
44
psutil==5.9.4
55
tqdm==4.64.1

src/ansys/dpf/core/collection.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,7 @@ def integral_collection(inpt, server: BaseServer = None):
105105
inpt = inpt.flatten()
106106
if all(isinstance(x, (int, np.int32)) for x in inpt):
107107
return IntCollection(inpt, server=server)
108-
if all(isinstance(x, (float, np.float)) for x in inpt):
108+
if all(isinstance(x, (float, np.float64)) for x in inpt):
109109
return FloatCollection(inpt, server=server)
110110
else:
111111
raise NotImplementedError(
@@ -556,7 +556,7 @@ def create_subtype(self, obj_by_copy):
556556
return float(obj_by_copy)
557557

558558
def _set_integral_entries(self, input):
559-
dtype = np.float
559+
dtype = np.float64
560560
if isinstance(input, range):
561561
input = np.array(list(input), dtype=dtype)
562562
elif not isinstance(input, (np.ndarray, np.generic)):

src/ansys/dpf/core/elements.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -656,15 +656,13 @@ def map_scoping(self, external_scope):
656656
>>> vol = model.results.elemental_volume()
657657
>>> field = vol.outputs.fields_container()[0]
658658
>>> ind, mask = elements.map_scoping(field.scoping)
659-
>>> ind
660-
array([0, 1, 2, 3, 4, 5, 6, 7])
661659
662660
"""
663661
if external_scope.location in ["Nodal", "NodalElemental"]:
664662
raise ValueError('Input scope location must be "Nodal"')
665663
arr = np.array(list(map(self.mapping_id_to_index.get, external_scope.ids)))
666664
mask = arr != None
667-
ind = arr[mask].astype(np.int)
665+
ind = arr[mask].astype(np.int32)
668666
return ind, mask
669667

670668
@property

src/ansys/dpf/core/nodes.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -322,7 +322,7 @@ def map_scoping(self, external_scope):
322322
raise ValueError('Input scope location must be "Nodal"')
323323
arr = np.array(list(map(self.mapping_id_to_index.get, external_scope.ids)))
324324
mask = arr != None
325-
ind = arr[mask].astype(np.int)
325+
ind = arr[mask].astype(np.int32)
326326
return ind, mask
327327

328328
def add_node(self, id, coordinates):

tests/test_custom_type_field.py

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -54,14 +54,14 @@ def test_create_custom_type_field_push_back(server_type):
5454

5555
@conftest.raises_for_servers_version_under("5.0")
5656
def test_set_get_data_pointer_custom_type_field(server_type):
57-
field = dpf.core.CustomTypeField(np.float, nentities=20, server=server_type)
57+
field = dpf.core.CustomTypeField(np.float64, nentities=20, server=server_type)
5858
field_def = dpf.core.FieldDefinition(server=server_type)
5959
field_def.dimensionality = dpf.core.Dimensionality({3}, dpf.core.natures.vector)
6060
field.field_definition = field_def
6161
scop = dpf.core.Scoping(ids=[1, 2, 3, 4], location="faces", server=server_type)
6262
field.scoping = scop
6363

64-
data = np.empty((24,), dtype=np.float)
64+
data = np.empty((24,), dtype=np.float64)
6565
for i in range(0, 24):
6666
data[i] = i
6767
field.data = data
@@ -85,7 +85,7 @@ def test_set_get_data_pointer_custom_type_field(server_type):
8585

8686
@conftest.raises_for_servers_version_under("5.0")
8787
def test_set_get_field_def_custom_type_field(server_type):
88-
field = dpf.core.CustomTypeField(np.float, nentities=20, server=server_type)
88+
field = dpf.core.CustomTypeField(np.float64, nentities=20, server=server_type)
8989
field_def = dpf.core.FieldDefinition(server=server_type)
9090
field_def.dimensionality = dpf.core.Dimensionality([3], dpf.core.natures.vector)
9191
field_def.location = core.locations.elemental
@@ -116,14 +116,14 @@ def test_set_get_field_def_custom_type_field(server_type):
116116

117117
@conftest.raises_for_servers_version_under("5.0")
118118
def test_mutable_data_custom_type_field(server_clayer):
119-
field = dpf.core.CustomTypeField(np.float, nentities=20, server=server_clayer)
119+
field = dpf.core.CustomTypeField(np.float64, nentities=20, server=server_clayer)
120120
field_def = dpf.core.FieldDefinition(server=server_clayer)
121121
field_def.dimensionality = dpf.core.Dimensionality({3}, dpf.core.natures.vector)
122122
field.field_definition = field_def
123123
scop = dpf.core.Scoping(ids=[1, 2, 3, 4], location="faces", server=server_clayer)
124124
field.scoping = scop
125125

126-
data = np.empty((24,), dtype=np.float)
126+
data = np.empty((24,), dtype=np.float64)
127127
for i in range(0, 24):
128128
data[i] = i
129129
field.data = data
@@ -153,14 +153,14 @@ def test_mutable_data_custom_type_field(server_clayer):
153153

154154
@conftest.raises_for_servers_version_under("5.0")
155155
def test_mutable_data_contiguous_custom_type_field(server_clayer):
156-
field = dpf.core.CustomTypeField(np.float, nentities=20, server=server_clayer)
156+
field = dpf.core.CustomTypeField(np.float64, nentities=20, server=server_clayer)
157157
field_def = dpf.core.FieldDefinition(server=server_clayer)
158158
field_def.dimensionality = dpf.core.Dimensionality([6], dpf.core.natures.vector)
159159
field.field_definition = field_def
160160
scop = dpf.core.Scoping(ids=[1, 2, 3, 4], location="faces", server=server_clayer)
161161
field.scoping = scop
162162

163-
data = np.empty((24,), dtype=np.float)
163+
data = np.empty((24,), dtype=np.float64)
164164
for i in range(0, 24):
165165
data[i] = i
166166
field.data = data
@@ -189,15 +189,14 @@ def test_mutable_data_contiguous_custom_type_field(server_clayer):
189189

190190
# not using a fixture on purpose: the instance of simple field SHOULD be owned by each test
191191
def get_float_field(server_clayer):
192-
field = dpf.core.CustomTypeField(np.float32, nentities=20, server=server_clayer)
193-
field = dpf.core.CustomTypeField(np.float, nentities=20, server=server_clayer)
192+
field = dpf.core.CustomTypeField(np.float64, nentities=20, server=server_clayer)
194193
field_def = dpf.core.FieldDefinition(server=server_clayer)
195194
field_def.dimensionality = dpf.core.Dimensionality({3}, dpf.core.natures.vector)
196195
field.field_definition = field_def
197196
scop = dpf.core.Scoping(ids=[1, 2, 3, 4], location="faces", server=server_clayer)
198197
field.scoping = scop
199198

200-
data = np.empty((24,), dtype=np.float)
199+
data = np.empty((24,), dtype=np.float64)
201200
for i in range(0, 24):
202201
data[i] = i
203202
field.data = data

tests/test_field.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ def test_create_field(server_type):
2525

2626
def test_empty_field(server_type):
2727
field = dpf.core.Field(server=server_type)
28-
assert np.allclose(field.data, np.empty((0,), dtype=np.float))
28+
assert np.allclose(field.data, np.empty((0,), dtype=np.float64))
2929
field = dpf.core.PropertyField(server=server_type)
3030
assert np.allclose(field.data, np.empty((0,), dtype=np.int32))
3131

@@ -925,7 +925,7 @@ def get_simple_field(server_clayer):
925925
scop = dpf.core.Scoping(ids=[1, 2, 3, 4], location="faces", server=server_clayer)
926926
field.scoping = scop
927927

928-
data = np.empty((24,), dtype=np.float)
928+
data = np.empty((24,), dtype=np.float64)
929929
for i in range(0, 24):
930930
data[i] = i
931931
field.data = data
@@ -995,7 +995,7 @@ def test_mutable_entity_data_contiguous_field(server_clayer):
995995
scop = dpf.core.Scoping(ids=[1, 2, 3, 4], location="faces", server=server_clayer)
996996
field.scoping = scop
997997

998-
data = np.empty((24,), dtype=np.float)
998+
data = np.empty((24,), dtype=np.float64)
999999
for i in range(0, 24):
10001000
data[i] = i
10011001
field.data = data

0 commit comments

Comments
 (0)