-
Notifications
You must be signed in to change notification settings - Fork 3.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Autodiff] Optimize and eliminate the Jacobian tensor for te.autodiff #6078
Merged
Merged
Changes from 19 commits
Commits
Show all changes
25 commits
Select commit
Hold shift + click to select a range
7f6c5fc
[Autodiff] Optimize and eliminate the Jacobian tensor for te.autodiff
yzhliu 5972033
fix lint
yzhliu 018f67b
Merge remote-tracking branch 'upstream/master' into opt-autodiff
yzhliu 2bd109c
fix clang-format
yzhliu 004ae0b
add comments and magic number
yzhliu 737cdf9
clang-lint
yzhliu a3e6b7e
Merge remote-tracking branch 'upstream/master' into opt-autodiff
yzhliu d158851
Merge remote-tracking branch 'upstream/master' into opt-autodiff
yzhliu f9f4c18
Merge remote-tracking branch 'upstream/master' into opt-autodiff
yzhliu 5951a94
address some comments
yzhliu 914de04
Merge remote-tracking branch 'upstream/master' into opt-autodiff
yzhliu 69ea436
remove FreeVarsVisitor
yzhliu e866720
fix constexpr lint
yzhliu d7178b3
Merge remote-tracking branch 'upstream/master' into opt-autodiff
yzhliu 6596d7d
fix lint
yzhliu e6c0b9b
fix lint
yzhliu 79172d8
add Map.Merge
yzhliu bb19a32
Merge remote-tracking branch 'upstream/master' into opt-autodiff
yzhliu 9731d0f
lint
yzhliu 47a5852
change Array::Concat & Map::Merge to global functions
yzhliu fd3a4a5
Merge remote-tracking branch 'upstream/master' into opt-autodiff
yzhliu 4ede7fb
fix lint
yzhliu 357510b
move functions to global
yzhliu c80ef74
static -> inline
yzhliu e5746ed
Merge remote-tracking branch 'upstream/master' into opt-autodiff
yzhliu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -956,6 +956,19 @@ class Array : public ObjectRef { | |
return static_cast<ArrayNode*>(data_.get()); | ||
} | ||
|
||
/*! | ||
* \brief Concat with another Array. It does not mutate the current one. | ||
* \param other Array to be concatenated. | ||
* @return The concatenated Array. Original Array is kept unchanged. | ||
*/ | ||
Array<T> Concat(const Array<T>& other) const { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Consider make it as a global function, which also enables copy on write on the lhs(this) |
||
Array<T> ret = *this; | ||
for (const auto& x : other) { | ||
ret.push_back(x); | ||
} | ||
return std::move(ret); | ||
} | ||
|
||
/*! \brief specify container node */ | ||
using ContainerType = ArrayNode; | ||
|
||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shall we make it as a global function instead of member? so it is not ambiguitous (that the result is a new map)