Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[JS/WebGPU] Preserve zero size input tensor dims. #19737

Merged
merged 12 commits into from
Mar 8, 2024
Merged

[JS/WebGPU] Preserve zero size input tensor dims. #19737

Changes from 1 commit
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
da6e9f5
Preserve zero size input tensor dims.
satyajandhyala Mar 1, 2024
ae01d11
Use adjustedAxis instead of axis
satyajandhyala Mar 1, 2024
2c0d67c
Refactor; add more tests.
satyajandhyala Mar 1, 2024
ce6a004
Moved the new concat test cases from concat.jsonc to concat_zero-size…
satyajandhyala Mar 1, 2024
51c83d7
Removed calculateOutputShape function.
satyajandhyala Mar 4, 2024
2bfc23e
Removed special case handling of all inputs zero-sized
satyajandhyala Mar 5, 2024
e8fb4d2
Use adjusted axis to validate.
satyajandhyala Mar 5, 2024
f19ece3
Rollback code allowing zero-sized input non-concat axes dims mismatch…
satyajandhyala Mar 6, 2024
c0fab59
Keep the variable name adjustedAxis
satyajandhyala Mar 6, 2024
ed048b0
Removed referenceInput.
satyajandhyala Mar 7, 2024
971b612
Fixed comment
satyajandhyala Mar 7, 2024
adfca56
Check if the axis is with in the range.
satyajandhyala Mar 8, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fixed comment
  • Loading branch information
@satyajandhyala
satyajandhyala committed Mar 7, 2024
commit 971b6121a97bcc7b6248ab8245813f8edbe26296
3 changes: 0 additions & 3 deletions js/web/lib/wasm/jsep/webgpu/ops/concat.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -138,10 +138,7 @@ const createConcatProgramInfo =
};

export const concat = (context: ComputeContext, attributes: ConcatAttributes): void => {
// find a none zero tensor as reference to determine the output shape
// choose input 0 as reference if all input tensors are zero-sized.
const inputs = context.inputs;

const inputShape = inputs[0].dims;
const adjustedAxis = attributes.axis + (attributes.axis < 0 ? inputShape.length : 0);
validateInputs(inputs, adjustedAxis);
Expand Down
Loading