Commitc3c3452

committed

Apply ruff/refurb preview rule FURB145

FURB145 Prefer `copy` method over slicing

1 parent2c1cf76 commitc3c3452Copy full SHA for c3c3452

File tree

-3

lines changed

nipy
- algorithms/utils/tests
  - test_pca_image.py
- core/utils
  - generators.py

-3

lines changed

Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
`@@ -302,7 +302,7 @@ def test_other_axes(data_dict):`
`302`	`302`	`assert_almost_equal(pos_dp['basis_vectors'],pos_p[bv_key])`
`303`	`303`	`assert_almost_equal(pos_dp['basis_projections'],img_bps.get_fdata())`
`304`	`304`	`# And we've replaced the expected axis`
`305`		`-exp_coords=in_coords[:]`
	`305`	`+exp_coords=in_coords.copy()`
`306`	`306`	`exp_coords[exp_coords.index(axis_name)]='PCA components'`
`307`	`307`	`assertimg_bps.axes.coord_names==tuple(exp_coords)`
`308`	`308`	`# If the affine is not diagonal, we'll get an error`
`@@ -323,6 +323,6 @@ def test_other_axes(data_dict):`
`323`	`323`	`pytest.raises(AxisError,pca_image,nd_img,axis_name)`
`324`	`324`	`foraxis_namein'kt':`
`325`	`325`	`p=pca_image(img,axis_name,ncomp=ncomp)`
`326`		`-exp_coords=in_coords[:]`
	`326`	`+exp_coords=in_coords.copy()`
`327`	`327`	`exp_coords[exp_coords.index(axis_name)]='PCA components'`
`328`	`328`	`assertp['basis_projections'].axes.coord_names==tuple(exp_coords)`

Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -182,7 +182,7 @@ def slice_generator(data, axis=0):`
`182`	`182`	`slice_template= [slice(0,s)forsindata.shape]`
`183`	`183`
`184`	`184`	`forninrange(nmax):`
`185`		`-slices=slice_template[:]`
	`185`	`+slices=slice_template.copy()`
`186`	`186`	`for (a,div,mod)inzip(axis,divs,mods):`
`187`	`187`	`x=int(n/div%mod)`
`188`	`188`	`slices[a]=x`

Comments

(0)