Skip to content
This repository has been archived by the owner on Nov 17, 2023. It is now read-only.

Upgrade: dmlc::optional<T> constructors and value() function #20560

Draft
wants to merge 12 commits into
base: master
Choose a base branch
from
2 changes: 1 addition & 1 deletion 3rdparty/dmlc-core
Submodule dmlc-core updated 1 files
+160 −24 include/dmlc/optional.h
2 changes: 1 addition & 1 deletion 3rdparty/tvm
Submodule tvm updated from efdac9 to 07e81c
2 changes: 1 addition & 1 deletion include/mxnet/tuple.h
Original file line number Diff line number Diff line change
Expand Up @@ -382,7 +382,7 @@ class Tuple {

protected:
// stack cache size
static const int kStackCache = 4;
static const int kStackCache = 8;
/*! \brief number of dimension of the tuple */
int ndim_{0};
/*! \brief number of cells allocated in data_heap_ */
Expand Down
4 changes: 2 additions & 2 deletions src/operator/numpy/random/np_choice_op.h
Original file line number Diff line number Diff line change
Expand Up @@ -46,8 +46,8 @@ struct NumpyChoiceParam : public dmlc::Parameter<NumpyChoiceParam> {
bool replace;
bool weighted;
DMLC_DECLARE_PARAMETER(NumpyChoiceParam) {
DMLC_DECLARE_FIELD(a);
DMLC_DECLARE_FIELD(size);
DMLC_DECLARE_FIELD(a).set_default(dmlc::optional<int64_t>());
DMLC_DECLARE_FIELD(size).set_default(dmlc::optional<mxnet::Tuple<int64_t>>());
DMLC_DECLARE_FIELD(ctx).set_default("cpu");
DMLC_DECLARE_FIELD(replace).set_default(true);
DMLC_DECLARE_FIELD(weighted).set_default(false);
Expand Down