This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

Add var->value() feature #18

Merged

kexinzhao merged 1 commit into PaddlePaddle:develop from kexinzhao:tape_eval_value

Jun 19, 2018

Collaborator

kexinzhao commented Jun 19, 2018

fix #11


          add value() feature

67d5344

kexinzhao requested a review from tonyyang-svail

June 19, 2018 21:18

tonyyang-svail approved these changes

View reviewed changes

src/variable.h

                 // TODO(tonyyang-svail): No need to expose name
                 std::string Name() const { return desc_.Name(); }
-                framework::Variable *Var() { return &var_; }
+                const framework::Variable& Var() const { return var_; }
+                framework::Variable* MutableVar() { return &var_; }

Collaborator

tonyyang-svail Jun 19, 2018

Good catch.

kexinzhao merged commit a3ce862 into PaddlePaddle:develop

kexinzhao deleted the tape_eval_value branch

June 19, 2018 21:23

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.