Feature #10455
[PATCH 0/n] Combine interface for creating new matrix
Status:
Open
Priority:
Normal
Assignee:
-
Target version:
-
Description
I believe we'd better reduce the number of interface for creating new matrix as much as possible.
Right now, we have 7 methods for creating new matrix,
* Matrix.[](*rows) * Matrix.rows(rows, copy = true) * Matrix.columns(columns) * Matrix.build(row_count, column_count, &block) * Matrix.row_vector(row) * Matrix.column_vector(column) * Matrix#new_matrix (for internal)
I know each methods has meaning for each specific way.
But I wanna combine Matrix.rows
with Matrix.row_vector
and
Matrix.columns
with Matrix.column_vector
.
I suppose only Matrix.rows
and Matrix.columns
is totally enough.
Matrix.row_vector(row)
is equivalent Matrix.rows([row])
and
Matrix.column_vector(column)
is equivalent Matrix.columns([column])
If it looks fine for you, I'll make patches.
Thanks.
Updated by gogotanaka (Kazuki Tanaka) about 6 years ago
For now, I'm gonna make Matrix.row_vector(row)
and Matrix.column_vector(column)
deprecated.