8000 Fix llm inference output by p018841p-coder · Pull Request #624 · google-ai-edge/mediapipe-samples · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@p018841p-coder
Copy link

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context.

Fixes # (issue)

Checklist

Please ensure the following items are complete before submitting a pull request:

  • My code follows the code style of the project.
  • I have updated the documentation (if applicable).
  • I have added tests to cover my changes.

Type of Change

Please check the relevant option below:

  • Bug fix (non-breaking change which fixes an issue)
  • Documentation update (non-breaking change which updates documentation)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Screenshots

If applicable, please add screenshots to help explain your changes.

Additional Notes

Add any additional information or context about the pull request here.

google-labs-jules bot and others added 2 commits October 3, 2025 20:35
The `examples/llm_inference/js/index.js` file was using `output.textContent` to set the value of a `<textarea>` element. The correct property for this is `output.value`.

This bug prevented the generated text from being displayed in the output text area. This commit replaces all instances of `output.textContent` with `output.value` to fix the issue.
@google-cla
Copy link
google-cla bot commented Oct 3, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

0