From 6d1529983f303150bc65bdfed2beeda54bc388c7 Mon Sep 17 00:00:00 2001 From: alex-yuyue Date: Tue, 29 Dec 2020 13:42:50 -0500 Subject: [PATCH] Fix CI warning, use CONST_POINTER in node build Signed-off-by: alex-yuyue --- .../ccsrc/minddata/dataset/engine/ir/datasetops/batch_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/batch_node.h | 2 +- .../dataset/engine/ir/datasetops/bucket_batch_by_length_node.cc | 2 +- .../dataset/engine/ir/datasetops/bucket_batch_by_length_node.h | 2 +- .../engine/ir/datasetops/build_sentence_piece_vocab_node.cc | 2 +- .../engine/ir/datasetops/build_sentence_piece_vocab_node.h | 2 +- .../minddata/dataset/engine/ir/datasetops/build_vocab_node.cc | 2 +- .../minddata/dataset/engine/ir/datasetops/build_vocab_node.h | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/concat_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/concat_node.h | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/dataset_node.h | 2 +- .../minddata/dataset/engine/ir/datasetops/epoch_ctrl_node.cc | 2 +- .../minddata/dataset/engine/ir/datasetops/epoch_ctrl_node.h | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/filter_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/filter_node.h | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/map_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/map_node.h | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/project_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/project_node.h | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/rename_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/rename_node.h | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/repeat_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/repeat_node.h | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/root_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/root_node.h | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/shuffle_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/shuffle_node.h | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/skip_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/skip_node.h | 2 +- .../minddata/dataset/engine/ir/datasetops/source/album_node.cc | 2 +- .../minddata/dataset/engine/ir/datasetops/source/album_node.h | 2 +- .../minddata/dataset/engine/ir/datasetops/source/celeba_node.cc | 2 +- .../minddata/dataset/engine/ir/datasetops/source/celeba_node.h | 2 +- .../dataset/engine/ir/datasetops/source/cifar100_node.cc | 2 +- .../dataset/engine/ir/datasetops/source/cifar100_node.h | 2 +- .../dataset/engine/ir/datasetops/source/cifar10_node.cc | 2 +- .../minddata/dataset/engine/ir/datasetops/source/cifar10_node.h | 2 +- .../minddata/dataset/engine/ir/datasetops/source/clue_node.cc | 2 +- .../minddata/dataset/engine/ir/datasetops/source/clue_node.h | 2 +- .../minddata/dataset/engine/ir/datasetops/source/coco_node.cc | 2 +- .../minddata/dataset/engine/ir/datasetops/source/coco_node.h | 2 +- .../minddata/dataset/engine/ir/datasetops/source/csv_node.cc | 2 +- .../minddata/dataset/engine/ir/datasetops/source/csv_node.h | 2 +- .../dataset/engine/ir/datasetops/source/generator_node.cc | 2 +- .../dataset/engine/ir/datasetops/source/generator_node.h | 2 +- .../dataset/engine/ir/datasetops/source/image_folder_node.cc | 2 +- .../dataset/engine/ir/datasetops/source/image_folder_node.h | 2 +- .../dataset/engine/ir/datasetops/source/manifest_node.cc | 2 +- .../dataset/engine/ir/datasetops/source/manifest_node.h | 2 +- .../dataset/engine/ir/datasetops/source/minddata_node.cc | 2 +- .../dataset/engine/ir/datasetops/source/minddata_node.h | 2 +- .../minddata/dataset/engine/ir/datasetops/source/mnist_node.cc | 2 +- .../minddata/dataset/engine/ir/datasetops/source/mnist_node.h | 2 +- .../minddata/dataset/engine/ir/datasetops/source/random_node.cc | 2 +- .../minddata/dataset/engine/ir/datasetops/source/random_node.h | 2 +- .../dataset/engine/ir/datasetops/source/text_file_node.cc | 2 +- .../dataset/engine/ir/datasetops/source/text_file_node.h | 2 +- .../dataset/engine/ir/datasetops/source/tf_record_node.cc | 2 +- .../dataset/engine/ir/datasetops/source/tf_record_node.h | 2 +- .../minddata/dataset/engine/ir/datasetops/source/voc_node.cc | 2 +- .../minddata/dataset/engine/ir/datasetops/source/voc_node.h | 2 +- .../minddata/dataset/engine/ir/datasetops/sync_wait_node.cc | 2 +- .../minddata/dataset/engine/ir/datasetops/sync_wait_node.h | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/take_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/take_node.h | 2 +- .../minddata/dataset/engine/ir/datasetops/transfer_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/transfer_node.h | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/zip_node.cc | 2 +- .../ccsrc/minddata/dataset/engine/ir/datasetops/zip_node.h | 2 +- 69 files changed, 69 insertions(+), 69 deletions(-) diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/batch_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/batch_node.cc index dfe2190125..588f45ace6 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/batch_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/batch_node.cc @@ -93,7 +93,7 @@ Status BatchNode::ValidateParams() { return Status::OK(); } -Status BatchNode::Build(std::vector> *node_ops) { +Status BatchNode::Build(std::vector> *const node_ops) { #ifdef ENABLE_PYTHON // if col_order_ isn't empty, then a project node needs to be attached after batch node. (same as map) // this means project_node needs to be the parent of batch_node. this means *node_ops = [project_node, batch_node] diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/batch_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/batch_node.h index ec39cd9733..b3f9613b2b 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/batch_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/batch_node.h @@ -60,7 +60,7 @@ class BatchNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/bucket_batch_by_length_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/bucket_batch_by_length_node.cc index c1cea3a8c1..c72faf666c 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/bucket_batch_by_length_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/bucket_batch_by_length_node.cc @@ -82,7 +82,7 @@ void BucketBatchByLengthNode::Print(std::ostream &out) const { out << ")"; } -Status BucketBatchByLengthNode::Build(std::vector> *node_ops) { +Status BucketBatchByLengthNode::Build(std::vector> *const node_ops) { bucket_boundaries_.insert(bucket_boundaries_.begin(), 0); node_ops->push_back(std::make_shared( column_names_, bucket_boundaries_, bucket_batch_sizes_, element_length_function_, pad_info_, diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/bucket_batch_by_length_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/bucket_batch_by_length_node.h index d163667671..1afe5a77c6 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/bucket_batch_by_length_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/bucket_batch_by_length_node.h @@ -55,7 +55,7 @@ class BucketBatchByLengthNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_sentence_piece_vocab_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_sentence_piece_vocab_node.cc index 4bfcdc5576..d643ac8607 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_sentence_piece_vocab_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_sentence_piece_vocab_node.cc @@ -54,7 +54,7 @@ void BuildSentenceVocabNode::Print(std::ostream &out) const { } // Function to build BuildSentenceVocabNode -Status BuildSentenceVocabNode::Build(std::vector> *node_ops) { +Status BuildSentenceVocabNode::Build(std::vector> *const node_ops) { std::shared_ptr build_sentence_piece_vocab_op; build_sentence_piece_vocab_op = std::make_shared( vocab_, col_names_, vocab_size_, character_coverage_, model_type_, params_, connector_que_size_); diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_sentence_piece_vocab_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_sentence_piece_vocab_node.h index 2426201fc5..3904b6cbb9 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_sentence_piece_vocab_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_sentence_piece_vocab_node.h @@ -53,7 +53,7 @@ class BuildSentenceVocabNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_vocab_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_vocab_node.cc index a4041ac7e3..d085c4c0fe 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_vocab_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_vocab_node.cc @@ -50,7 +50,7 @@ void BuildVocabNode::Print(std::ostream &out) const { } // Function to build BuildVocabNode -Status BuildVocabNode::Build(std::vector> *node_ops) { +Status BuildVocabNode::Build(std::vector> *const node_ops) { std::shared_ptr build_vocab_op; build_vocab_op = std::make_shared(vocab_, columns_, freq_range_, top_k_, special_tokens_, special_first_, num_workers_, connector_que_size_); diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_vocab_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_vocab_node.h index 77193faf73..62e1eaff3b 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_vocab_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/build_vocab_node.h @@ -52,7 +52,7 @@ class BuildVocabNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/concat_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/concat_node.cc index bf72287682..d255242ac2 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/concat_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/concat_node.cc @@ -72,7 +72,7 @@ Status ConcatNode::ValidateParams() { return Status::OK(); } -Status ConcatNode::Build(std::vector> *node_ops) { +Status ConcatNode::Build(std::vector> *const node_ops) { if (children_flag_and_nums_.empty() || children_start_end_index_.empty()) { node_ops->push_back(std::make_shared(connector_que_size_)); } else { diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/concat_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/concat_node.h index 694445939d..5d5f2239dd 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/concat_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/concat_node.h @@ -53,7 +53,7 @@ class ConcatNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/dataset_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/dataset_node.h index a4c3459d80..9aef52bf40 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/dataset_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/dataset_node.h @@ -148,7 +148,7 @@ class DatasetNode : public std::enable_shared_from_this { /// \brief Pure virtual function to convert a DatasetNode class into a runtime dataset object /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - virtual Status Build(std::vector> *node_ops) = 0; + virtual Status Build(std::vector> *const node_ops) = 0; /// \brief base virtual function for derived class to implement parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/epoch_ctrl_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/epoch_ctrl_node.cc index 8bea432e5b..f05adbb2aa 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/epoch_ctrl_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/epoch_ctrl_node.cc @@ -41,7 +41,7 @@ std::shared_ptr EpochCtrlNode::Copy() { void EpochCtrlNode::Print(std::ostream &out) const { out << Name() + "(epoch:" + std::to_string(num_epochs_) + ")"; } // Function to build the EpochCtrlOp -Status EpochCtrlNode::Build(std::vector> *node_ops) { +Status EpochCtrlNode::Build(std::vector> *const node_ops) { node_ops->push_back(std::make_shared(num_epochs_)); return Status::OK(); } diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/epoch_ctrl_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/epoch_ctrl_node.h index 046561fb12..78fda0c778 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/epoch_ctrl_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/epoch_ctrl_node.h @@ -52,7 +52,7 @@ class EpochCtrlNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/filter_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/filter_node.cc index 2da7e15b39..5da04af1b1 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/filter_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/filter_node.cc @@ -43,7 +43,7 @@ void FilterNode::Print(std::ostream &out) const { out << Name() + "(," + "input_cols:" + PrintColumns(input_columns_) + ")"; } -Status FilterNode::Build(std::vector> *node_ops) { +Status FilterNode::Build(std::vector> *const node_ops) { node_ops->push_back(std::make_shared(input_columns_, num_workers_, connector_que_size_, predicate_)); return Status::OK(); } diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/filter_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/filter_node.h index 88cf0323f2..68e3c45df0 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/filter_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/filter_node.h @@ -50,7 +50,7 @@ class FilterNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/map_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/map_node.cc index e010e8b498..e2e1fe073b 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/map_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/map_node.cc @@ -54,7 +54,7 @@ void MapNode::Print(std::ostream &out) const { << operations_.size() << ",...)"; } -Status MapNode::Build(std::vector> *node_ops) { +Status MapNode::Build(std::vector> *const node_ops) { std::vector> tensor_ops; // Build tensorOp from tensorOperation vector diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/map_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/map_node.h index 9a23d58370..59a8b1a4b8 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/map_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/map_node.h @@ -52,7 +52,7 @@ class MapNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/project_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/project_node.cc index 66b8acc1dd..50174d87bf 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/project_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/project_node.cc @@ -52,7 +52,7 @@ Status ProjectNode::ValidateParams() { return Status::OK(); } -Status ProjectNode::Build(std::vector> *node_ops) { +Status ProjectNode::Build(std::vector> *const node_ops) { node_ops->push_back(std::make_shared(columns_)); return Status::OK(); } diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/project_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/project_node.h index ee830551e0..de77bd997c 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/project_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/project_node.h @@ -49,7 +49,7 @@ class ProjectNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/rename_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/rename_node.cc index 31cc7ca1e4..64b924b53b 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/rename_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/rename_node.cc @@ -57,7 +57,7 @@ Status RenameNode::ValidateParams() { return Status::OK(); } -Status RenameNode::Build(std::vector> *node_ops) { +Status RenameNode::Build(std::vector> *const node_ops) { node_ops->push_back(std::make_shared(input_columns_, output_columns_, connector_que_size_)); return Status::OK(); } diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/rename_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/rename_node.h index 3db1557e92..2fca8c7966 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/rename_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/rename_node.h @@ -50,7 +50,7 @@ class RenameNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/repeat_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/repeat_node.cc index adc0f05356..cd4b4109ed 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/repeat_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/repeat_node.cc @@ -37,7 +37,7 @@ std::shared_ptr RepeatNode::Copy() { void RepeatNode::Print(std::ostream &out) const { out << Name() + "(count:" + std::to_string(repeat_count_) + ")"; } -Status RepeatNode::Build(std::vector> *node_ops) { +Status RepeatNode::Build(std::vector> *const node_ops) { node_ops->push_back(std::make_shared(repeat_count_)); return Status::OK(); } diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/repeat_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/repeat_node.h index 4c549e2ccd..e5eeb173b0 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/repeat_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/repeat_node.h @@ -51,7 +51,7 @@ class RepeatNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/root_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/root_node.cc index 31123b709e..0f116d85e9 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/root_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/root_node.cc @@ -40,7 +40,7 @@ std::shared_ptr RootNode::Copy() { void RootNode::Print(std::ostream &out) const { out << Name(); } -Status RootNode::Build(std::vector> *node_ops) { +Status RootNode::Build(std::vector> *const node_ops) { // root node doesn't build a runtime Op. this function should return Status::Error when called. std::string err_msg = "Root node doesn't build a runtime Op"; MS_LOG(ERROR) << err_msg; diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/root_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/root_node.h index a6a1ea2b59..932c5439c6 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/root_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/root_node.h @@ -52,7 +52,7 @@ class RootNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Getter of number of epochs int32_t num_epochs() { return num_epochs_; } diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/shuffle_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/shuffle_node.cc index b5bdc203ec..5592d5e559 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/shuffle_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/shuffle_node.cc @@ -43,7 +43,7 @@ void ShuffleNode::Print(std::ostream &out) const { } // Function to build the ShuffleOp -Status ShuffleNode::Build(std::vector> *node_ops) { +Status ShuffleNode::Build(std::vector> *const node_ops) { node_ops->push_back(std::make_shared(shuffle_size_, shuffle_seed_, connector_que_size_, reset_every_epoch_, rows_per_buffer_)); return Status::OK(); diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/shuffle_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/shuffle_node.h index 6f70391770..5083249523 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/shuffle_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/shuffle_node.h @@ -49,7 +49,7 @@ class ShuffleNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; Status ValidateParams() override; diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/skip_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/skip_node.cc index b2db03902c..a5e6d74ba4 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/skip_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/skip_node.cc @@ -38,7 +38,7 @@ std::shared_ptr SkipNode::Copy() { void SkipNode::Print(std::ostream &out) const { out << Name() + "(skip_count:" + std::to_string(skip_count_) + ")"; } // Function to build the SkipOp -Status SkipNode::Build(std::vector> *node_ops) { +Status SkipNode::Build(std::vector> *const node_ops) { node_ops->push_back(std::make_shared(skip_count_, connector_que_size_)); return Status::OK(); } diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/skip_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/skip_node.h index ba73cf032a..61a8db37f9 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/skip_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/skip_node.h @@ -49,7 +49,7 @@ class SkipNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/album_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/album_node.cc index 924c351c10..ac527e6d26 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/album_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/album_node.cc @@ -65,7 +65,7 @@ Status AlbumNode::ValidateParams() { } // Function to build AlbumNode -Status AlbumNode::Build(std::vector> *node_ops) { +Status AlbumNode::Build(std::vector> *const node_ops) { auto schema = std::make_unique(); RETURN_IF_NOT_OK(schema->LoadSchemaFile(schema_path_, column_names_)); diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/album_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/album_node.h index 6f1f614e1c..258d1dde9c 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/album_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/album_node.h @@ -51,7 +51,7 @@ class AlbumNode : public MappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/celeba_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/celeba_node.cc index cd5638c163..4ec79a4c0c 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/celeba_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/celeba_node.cc @@ -61,7 +61,7 @@ Status CelebANode::ValidateParams() { } // Function to build CelebANode -Status CelebANode::Build(std::vector> *node_ops) { +Status CelebANode::Build(std::vector> *const node_ops) { std::unique_ptr schema = std::make_unique(); RETURN_IF_NOT_OK(schema->AddColumn(ColDescriptor("image", DataType(DataType::DE_UINT8), TensorImpl::kFlexible, 1))); // label is like this:0 1 0 0 1...... diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/celeba_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/celeba_node.h index a3556ef1b1..79cbdeabfd 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/celeba_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/celeba_node.h @@ -52,7 +52,7 @@ class CelebANode : public MappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar100_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar100_node.cc index c4e1643c49..8ea425a913 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar100_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar100_node.cc @@ -54,7 +54,7 @@ Status Cifar100Node::ValidateParams() { } // Function to build CifarOp for Cifar100 -Status Cifar100Node::Build(std::vector> *node_ops) { +Status Cifar100Node::Build(std::vector> *const node_ops) { // Do internal Schema generation. auto schema = std::make_unique(); RETURN_IF_NOT_OK(schema->AddColumn(ColDescriptor("image", DataType(DataType::DE_UINT8), TensorImpl::kCv, 1))); diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar100_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar100_node.h index 2fb7b3efce..a18fa827d8 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar100_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar100_node.h @@ -50,7 +50,7 @@ class Cifar100Node : public MappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar10_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar10_node.cc index 979893bb90..d5afadcdb3 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar10_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar10_node.cc @@ -54,7 +54,7 @@ Status Cifar10Node::ValidateParams() { } // Function to build CifarOp for Cifar10 -Status Cifar10Node::Build(std::vector> *node_ops) { +Status Cifar10Node::Build(std::vector> *const node_ops) { // Do internal Schema generation. auto schema = std::make_unique(); RETURN_IF_NOT_OK(schema->AddColumn(ColDescriptor("image", DataType(DataType::DE_UINT8), TensorImpl::kCv, 1))); diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar10_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar10_node.h index 5e71518a37..2ea179856a 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar10_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/cifar10_node.h @@ -50,7 +50,7 @@ class Cifar10Node : public MappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/clue_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/clue_node.cc index 570962d5b8..6320f39cf0 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/clue_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/clue_node.cc @@ -84,7 +84,7 @@ std::vector CLUENode::split(const std::string &s, char delim) { } // Function to build CLUENode -Status CLUENode::Build(std::vector> *node_ops) { +Status CLUENode::Build(std::vector> *const node_ops) { std::map key_map; if (task_ == "AFQMC") { if (usage_ == "train") { diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/clue_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/clue_node.h index 13953126f5..1d78504b7f 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/clue_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/clue_node.h @@ -52,7 +52,7 @@ class CLUENode : public NonMappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/coco_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/coco_node.cc index e4fd99c4ab..cae79ab334 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/coco_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/coco_node.cc @@ -64,7 +64,7 @@ Status CocoNode::ValidateParams() { } // Function to build CocoNode -Status CocoNode::Build(std::vector> *node_ops) { +Status CocoNode::Build(std::vector> *const node_ops) { CocoOp::TaskType task_type = CocoOp::TaskType::Detection; if (task_ == "Detection") { task_type = CocoOp::TaskType::Detection; diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/coco_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/coco_node.h index 5408bc42ba..b431f6394b 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/coco_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/coco_node.h @@ -50,7 +50,7 @@ class CocoNode : public MappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/csv_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/csv_node.cc index b8dc3243e9..8d3c0f0f30 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/csv_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/csv_node.cc @@ -92,7 +92,7 @@ Status CSVNode::ValidateParams() { } // Function to build CSVNode -Status CSVNode::Build(std::vector> *node_ops) { +Status CSVNode::Build(std::vector> *const node_ops) { bool shuffle_files = (shuffle_ == ShuffleMode::kGlobal || shuffle_ == ShuffleMode::kFiles); // CSVOp by itself is a non-mappable dataset that does not support sampling. diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/csv_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/csv_node.h index 61e4188fab..490f842ce1 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/csv_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/csv_node.h @@ -73,7 +73,7 @@ class CSVNode : public NonMappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/generator_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/generator_node.cc index 5c7bed1109..3e1449cac4 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/generator_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/generator_node.cc @@ -50,7 +50,7 @@ void GeneratorNode::Print(std::ostream &out) const { out << Name() + "(:" + ",columns:" + PrintColumns(column_names_) + ",)"; } -Status GeneratorNode::Build(std::vector> *node_ops) { +Status GeneratorNode::Build(std::vector> *const node_ops) { std::unique_ptr data_schema = std::make_unique(); if (schema_ != nullptr) { diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/generator_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/generator_node.h index 1c2c761c83..5c921344cb 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/generator_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/generator_node.h @@ -55,7 +55,7 @@ class GeneratorNode : public MappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/image_folder_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/image_folder_node.cc index 0448672864..7034f593cc 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/image_folder_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/image_folder_node.cc @@ -61,7 +61,7 @@ Status ImageFolderNode::ValidateParams() { return Status::OK(); } -Status ImageFolderNode::Build(std::vector> *node_ops) { +Status ImageFolderNode::Build(std::vector> *const node_ops) { // Do internal Schema generation. // This arg is exist in ImageFolderOp, but not externalized (in Python API). std::unique_ptr schema = std::make_unique(); diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/image_folder_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/image_folder_node.h index 236276644d..7c0c113525 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/image_folder_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/image_folder_node.h @@ -56,7 +56,7 @@ class ImageFolderNode : public MappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/manifest_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/manifest_node.cc index 7b033efbf5..768e83707d 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/manifest_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/manifest_node.cc @@ -82,7 +82,7 @@ Status ManifestNode::ValidateParams() { return Status::OK(); } -Status ManifestNode::Build(std::vector> *node_ops) { +Status ManifestNode::Build(std::vector> *const node_ops) { // Do internal Schema generation. auto schema = std::make_unique(); RETURN_IF_NOT_OK(schema->AddColumn(ColDescriptor("image", DataType(DataType::DE_UINT8), TensorImpl::kCv, 1))); diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/manifest_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/manifest_node.h index 1c1e109866..4ebbff9cc2 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/manifest_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/manifest_node.h @@ -51,7 +51,7 @@ class ManifestNode : public MappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/minddata_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/minddata_node.cc index 592f06ff98..f4ca3503aa 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/minddata_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/minddata_node.cc @@ -150,7 +150,7 @@ Status MindDataNode::BuildMindDatasetSamplerChain(const std::shared_ptr *sample_bytes) { sample_bytes_ = *sample_bytes; } -Status MindDataNode::Build(std::vector> *node_ops) { +Status MindDataNode::Build(std::vector> *const node_ops) { RETURN_IF_NOT_OK(BuildMindDatasetSamplerChain(sampler_, &operators_, num_padded_)); std::shared_ptr mindrecord_op; diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/minddata_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/minddata_node.h index c78d95ec99..1503baf9d1 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/minddata_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/minddata_node.h @@ -55,7 +55,7 @@ class MindDataNode : public MappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/mnist_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/mnist_node.cc index 1b7471a53f..0a7f7b60ec 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/mnist_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/mnist_node.cc @@ -50,7 +50,7 @@ Status MnistNode::ValidateParams() { return Status::OK(); } -Status MnistNode::Build(std::vector> *node_ops) { +Status MnistNode::Build(std::vector> *const node_ops) { // Do internal Schema generation. auto schema = std::make_unique(); RETURN_IF_NOT_OK(schema->AddColumn(ColDescriptor("image", DataType(DataType::DE_UINT8), TensorImpl::kCv, 1))); diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/mnist_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/mnist_node.h index 30b7d05a04..345acce177 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/mnist_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/mnist_node.h @@ -50,7 +50,7 @@ class MnistNode : public MappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/random_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/random_node.cc index 27c3743d2e..840d3082bb 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/random_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/random_node.cc @@ -67,7 +67,7 @@ int32_t RandomNode::GenRandomInt(int32_t min, int32_t max) { } // Build for RandomNode -Status RandomNode::Build(std::vector> *node_ops) { +Status RandomNode::Build(std::vector> *const node_ops) { rand_gen_.seed(GetSeed()); // seed the random generator // If total rows was not given, then randomly pick a number std::shared_ptr schema_obj; diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/random_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/random_node.h index a18c193dd3..1894441896 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/random_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/random_node.h @@ -70,7 +70,7 @@ class RandomNode : public NonMappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/text_file_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/text_file_node.cc index 9e168b94b0..136dbb23a4 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/text_file_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/text_file_node.cc @@ -70,7 +70,7 @@ Status TextFileNode::ValidateParams() { } // Function to build TextFileNode -Status TextFileNode::Build(std::vector> *node_ops) { +Status TextFileNode::Build(std::vector> *const node_ops) { bool shuffle_files = (shuffle_ == ShuffleMode::kGlobal || shuffle_ == ShuffleMode::kFiles); // TextFileOp by itself is a non-mappable dataset that does not support sampling. diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/text_file_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/text_file_node.h index 7dfb214841..94c0afe863 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/text_file_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/text_file_node.h @@ -52,7 +52,7 @@ class TextFileNode : public NonMappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/tf_record_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/tf_record_node.cc index a03aa77e1e..75608b51b2 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/tf_record_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/tf_record_node.cc @@ -105,7 +105,7 @@ Status TFRecordNode::ValidateParams() { } // Function to build TFRecordNode -Status TFRecordNode::Build(std::vector> *node_ops) { +Status TFRecordNode::Build(std::vector> *const node_ops) { // Sort the datasets file in a lexicographical order std::vector sorted_dir_files = dataset_files_; std::sort(sorted_dir_files.begin(), sorted_dir_files.end()); diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/tf_record_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/tf_record_node.h index d985727d0a..7716f645fb 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/tf_record_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/tf_record_node.h @@ -85,7 +85,7 @@ class TFRecordNode : public NonMappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/voc_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/voc_node.cc index 2a66e29bea..81ee00311f 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/voc_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/voc_node.cc @@ -85,7 +85,7 @@ Status VOCNode::ValidateParams() { } // Function to build VOCNode -Status VOCNode::Build(std::vector> *node_ops) { +Status VOCNode::Build(std::vector> *const node_ops) { auto schema = std::make_unique(); VOCOp::TaskType task_type_; diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/voc_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/voc_node.h index a3ab01fbbb..ba4df831fa 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/voc_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/source/voc_node.h @@ -52,7 +52,7 @@ class VOCNode : public MappableSourceNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/sync_wait_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/sync_wait_node.cc index 7013acc7a8..fdb7ae45be 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/sync_wait_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/sync_wait_node.cc @@ -42,7 +42,7 @@ void SyncWaitNode::Print(std::ostream &out) const { } // Function to build the BarrierOp -Status SyncWaitNode::Build(std::vector> *node_ops) { +Status SyncWaitNode::Build(std::vector> *const node_ops) { // Right now barrier should only take num_rows_per_buffer = 1 // The reason for this is because having it otherwise can lead to blocking issues // See barrier_op.h for more details diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/sync_wait_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/sync_wait_node.h index 1f2d793045..cd1a07304b 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/sync_wait_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/sync_wait_node.h @@ -51,7 +51,7 @@ class SyncWaitNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/take_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/take_node.cc index 2b15ede536..62a98c86bd 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/take_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/take_node.cc @@ -39,7 +39,7 @@ std::shared_ptr TakeNode::Copy() { void TakeNode::Print(std::ostream &out) const { out << Name() + "(num_rows:" + std::to_string(take_count_) + ")"; } // Function to build the TakeOp -Status TakeNode::Build(std::vector> *node_ops) { +Status TakeNode::Build(std::vector> *const node_ops) { node_ops->push_back(std::make_shared(take_count_, connector_que_size_)); return Status::OK(); } diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/take_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/take_node.h index 48bbea00bc..9760558c36 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/take_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/take_node.h @@ -49,7 +49,7 @@ class TakeNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/transfer_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/transfer_node.cc index f41180a8d7..8d48d66602 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/transfer_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/transfer_node.cc @@ -67,7 +67,7 @@ Status TransferNode::ValidateParams() { } // Function to build TransferNode -Status TransferNode::Build(std::vector> *node_ops) { +Status TransferNode::Build(std::vector> *const node_ops) { if (queue_name_.empty()) { // Get a uuid for queue name queue_name_ = Services::GetUniqueID(); diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/transfer_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/transfer_node.h index 0fac8bcf1a..603333f16e 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/transfer_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/transfer_node.h @@ -50,7 +50,7 @@ class TransferNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/zip_node.cc b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/zip_node.cc index 2c525ca179..12543d908f 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/zip_node.cc +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/zip_node.cc @@ -56,7 +56,7 @@ Status ZipNode::ValidateParams() { return Status::OK(); } -Status ZipNode::Build(std::vector> *node_ops) { +Status ZipNode::Build(std::vector> *const node_ops) { node_ops->push_back(std::make_shared(rows_per_buffer_, connector_que_size_)); return Status::OK(); } diff --git a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/zip_node.h b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/zip_node.h index a3e64a9401..06c8a084e2 100644 --- a/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/zip_node.h +++ b/mindspore/ccsrc/minddata/dataset/engine/ir/datasetops/zip_node.h @@ -49,7 +49,7 @@ class ZipNode : public DatasetNode { /// \brief a base class override function to create the required runtime dataset op objects for this class /// \param node_ops - A vector containing shared pointer to the Dataset Ops that this object will create /// \return Status Status::OK() if build successfully - Status Build(std::vector> *node_ops) override; + Status Build(std::vector> *const node_ops) override; /// \brief Parameters validation /// \return Status Status::OK() if all the parameters are valid