NNotepad: Add ability to show additional tensors in the output#311
Merged
Honry merged 1 commit intowebmachinelearning:masterfrom Mar 25, 2025
inexorabletash:nnotepad-multiple-outputs
Merged
NNotepad: Add ability to show additional tensors in the output#311Honry merged 1 commit intowebmachinelearning:masterfrom inexorabletash:nnotepad-multiple-outputs
Honry merged 1 commit intowebmachinelearning:masterfrom
inexorabletash:nnotepad-multiple-outputs
Conversation
Adds `output()` helper which can be used to list an identifier (or multiple identifiers) that will be shown in the results pane before the result of the final expression or assignment. A few notes: - Ending with an `output()` is supported, as a convenience. - The `output()` helper takes an identifier, not an expression. In the future this could change, either to taking an expression or showing the identifier name in the results pane. - Mixing `split()` (which returns a list of tensors) and `output()` is probably a bit confusing, since split already gives you multiple tensors, you're gonna have to pay attention to how many are coming from each. Sorry! Also fixes a minor issue with the call to asyncInit in js/tests.js - it wasn't awaited, so if the first test case was a test with a call it would fail.
Contributor
Author
|
@Honry - please take a look? Feature request from @philloooo |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Adds
output()helper which can be used to list an identifier (or multiple identifiers) that will be shown in the results pane before the result of the final expression or assignment.A few notes:
Ending with an
output()is supported, as a convenience.The
output()helper takes an identifier, not an expression. In the future this could change, either to taking an expression or showing the identifier name in the results pane.Mixing
split()(which returns a list of tensors) andoutput()is probably a bit confusing, since split already gives you multiple tensors, you're gonna have to pay attention to how many are coming from each. Sorry!Also fixes a minor issue with the call to asyncInit in js/tests.js - it wasn't awaited, so if the first test case was a test with a call it would fail.