singa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [singa] joddiy edited a comment on issue #696: Refactor autograd module
Date Thu, 14 May 2020 10:51:32 GMT

joddiy edited a comment on issue #696:
URL: https://github.com/apache/singa/issues/696#issuecomment-628554400


   > I see the @joddiy proposal is different only on a naming of a method, `compile` to
`construt_graph` in terms of API perspective. Maybe can upgrade the backend to graph construction
later without breaking new API?
   
   Hi, @dcslin , the key point is the `_call_` function of Operation(or Layer, since we want
to merge these two), this `_call` doesn't call the forward function instead it creates a new
object we called `Node`, so within the `construt_graph`, or we say, `compile`, we build the
graph with a placeholder, which means we don't need to postpone the graph construction til
we call forward function after we have the real data.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



Mime
View raw message