Skip to content

Commit a3aedc7

Browse files
committed
Addressed @matthew-brett's comments
1 parent 4107401 commit a3aedc7

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

nibabel/streamlines/array_sequence.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -387,7 +387,7 @@ def concatenate(seqs, axis):
387387
388388
Parameters
389389
----------
390-
seqs: list of :class:`ArraySequence` objects
390+
seqs: iterable of :class:`ArraySequence` objects
391391
Sequences to concatenate.
392392
axis : int
393393
Axis along which the sequences will be concatenated.

nibabel/streamlines/tests/test_array_sequence.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -305,12 +305,12 @@ def test_concatenate():
305305
seq = SEQ_DATA['seq'].copy() # In case there is in-place modification.
306306
seqs = [seq[:, [i]] for i in range(seq.common_shape[0])]
307307
new_seq = concatenate(seqs, axis=1)
308+
seq._data += 100 # Modifying the 'seq' shouldn't change 'new_seq'.
308309
check_arr_seq(new_seq, SEQ_DATA['data'])
309310
assert_true(not new_seq._is_view)
310311

311-
seq = SEQ_DATA['seq'].copy() # In case there is in-place modification.
312+
seq = SEQ_DATA['seq']
312313
seqs = [seq[:, [i]] for i in range(seq.common_shape[0])]
313314
new_seq = concatenate(seqs, axis=0)
314-
315315
assert_true(len(new_seq), seq.common_shape[0]*len(seq))
316316
assert_array_equal(new_seq._data, seq._data.T.reshape((-1, 1)))

0 commit comments

Comments
 (0)