Skip to content

Conversation

veblush
Copy link
Collaborator

@veblush veblush commented Sep 25, 2025

This is a temporary workaround to resolve a test failure on the main branch. This change should be reverted once the underlying issues are fixed and the tests are re-enabled.

The following tests are failing when executed with qemu-arm:

  • TestRecordsPersistentTfLiteTensorData
  • TestRecordsPersistentTfLiteTensorQuantizationData

Repro: (From the TFLM root directory)

$ docker run -it --rm -v $(pwd):/opt/tflm -w /opt/tflm ghcr.io/tflm-bot/tflm-ci:0.6.5  /bin/bash
# tensorflow/lite/micro/tools/ci_build/test_cortex_m_qemu.sh
# qemu-arm -cpu cortex-m3 gen/cortex_m_qemu_cortex-m3_default_cmsis_nn_gcc/bin/recording_micro_allocator_test

BUG=QuickWorkaroundOnMain

@veblush veblush added the ci:run label Sep 25, 2025
@TFLM-bot TFLM-bot removed the ci:run label Sep 25, 2025
@veblush veblush requested a review from gonnet September 26, 2025 00:06
@veblush veblush marked this pull request as ready for review September 26, 2025 00:06
@veblush veblush requested a review from a team as a code owner September 26, 2025 00:06
// "qemu: uncaught target signal 7 (Bus error) - core dumped"
#if 0
TF_LITE_MICRO_TEST(TestRecordsPersistentTfLiteTensorData) {
const tflite::Model* model = tflite::GetModel(kTestConvModelData);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@veblush I am suspicious of the existing kTestConvModelData (a generated convolutional model). The model we are using is from 2021, and when regenerated using the repo script, generates a very different model today. In my PR for schema buffer and custom option offsets, I generated a new model and placed it into the PR.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the pointer. I also concluded that the pre-generated test model file is misaligned, which is the culprit. I'm regenerating it in a separate PR to confirm this.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See #3207

@mergify mergify bot added the queued label Sep 26, 2025
mergify bot added a commit that referenced this pull request Sep 26, 2025
Copy link
Contributor

mergify bot commented Sep 26, 2025

This pull request has been removed from the queue for the following reason: checks timeout.

The merge conditions cannot be satisfied due to checks timeout.

You can check the last failing draft PR here: #3208.

You may have to fix your CI before adding the pull request to the queue again.

If you update this pull request, to fix the CI, it will automatically be requeued once the queue conditions match again.
If you think this was a flaky issue instead, you can requeue the pull request, without updating it, by posting a @mergifyio requeue comment.

@mergify mergify bot removed the queued label Sep 26, 2025
@veblush veblush merged commit 95a4591 into tensorflow:main Sep 26, 2025
106 of 110 checks passed
Laur59 added a commit to Laur59/tflite-micro that referenced this pull request Sep 30, 2025
95a4591	 Disabled two tests for qemutest (tensorflow#3206)	veblush@google.com	26.09.2025 07:12 PM
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.

4 participants