Skip to content

Integrate Common Inference Library, Optimize Memory Usage, and Enhance Display Configuration#13

Merged
ReinhardKeil merged 1 commit into
Arm-Examples:mainfrom
Jithendar11:feature/extend-model-support
Jul 6, 2026
Merged

Integrate Common Inference Library, Optimize Memory Usage, and Enhance Display Configuration#13
ReinhardKeil merged 1 commit into
Arm-Examples:mainfrom
Jithendar11:feature/extend-model-support

Conversation

@Jithendar11

Copy link
Copy Markdown
Contributor

Summary of changes:

  • Add support for multiple ML model categories, including classification and detection
  • Integrate the Common Inference Library (CIL) for ML model preprocessing and postprocessing
  • Update linker scripts to allocate memory for the CPL library
  • Increase LCD display resolution to 480x480
  • Configure camera resolution to 640x480 to optimize SRAM usage
  • Removed unused files

@Jithendar11 Jithendar11 force-pushed the feature/extend-model-support branch 2 times, most recently from 087233c to b0adc0a Compare July 6, 2026 05:26
@MiloradCvjetkovic

Copy link
Copy Markdown
Contributor

Hi @Jithendar11 ,

we have updated board layer and ExecuTorch pack, can you please rebase this pull-request to the current main branch.

Also, can you add cil.a library file as this one is not in your pull-request most likely because it was ignored due to settings in the .gitignore file.

@Jithendar11 Jithendar11 force-pushed the feature/extend-model-support branch from b0adc0a to 60bd6fe Compare July 6, 2026 11:20
…e Display Configuration

Summary of changes:
- Integrate the Common Inference Library (CIL) for ML model preprocessing and postprocessing
- Update linker scripts to allocate memory for the CIL library
- Increase LCD display resolution to 480x480
- Configure camera resolution to 640x480 to optimize SRAM usage
- Remove unused files
@Jithendar11 Jithendar11 force-pushed the feature/extend-model-support branch from 60bd6fe to 69c33e3 Compare July 6, 2026 11:33
@Jithendar11

Copy link
Copy Markdown
Contributor Author

Hi @MiloradCvjetkovic ,
Thanks for the review. I've addressed the requested changes by rebasing the pull request to the latest main branch and adding the cil.a library file. Could you please review it again?

@MiloradCvjetkovic

Copy link
Copy Markdown
Contributor

Hi @Jithendar11 ,

I have a few observations/questions:

  • could you avoid force-pushing and instead add new commits? It makes it much easier to track what changed, and the commits will be squashed when the PR is merged anyway.
  • could you remove cxx_stubs.cpp if it's no longer necessary?
  • is performance monitor not desired anymore?
  • can you keep the original comment line about Default value // <i> Default: 1 in the \Board\AppKit-E8_M55_HP\RTE\Device\AE822FA0E5597BS0_M55_HP\RTE_Device.h file
  • can you update SDS_Recordings directory with changes introduced? For example Camera parameters have changed, output data has changed, recordings are outdated and not in sync with actual firmware.
  • is there a reason Common Inference library is distributed in library form?

@ReinhardKeil ReinhardKeil self-requested a review July 6, 2026 13:13
@ReinhardKeil ReinhardKeil merged commit 4b8bf8d into Arm-Examples:main Jul 6, 2026
12 checks passed
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.

3 participants