Skip to content

Remove unused files and add default comments to AppKit-E8 RTE_Device.h#16

Merged
MiloradCvjetkovic merged 1 commit into
Arm-Examples:mainfrom
Jithendar11:chore/remove-unwanted-files
Jul 7, 2026
Merged

Remove unused files and add default comments to AppKit-E8 RTE_Device.h#16
MiloradCvjetkovic merged 1 commit into
Arm-Examples:mainfrom
Jithendar11:chore/remove-unwanted-files

Conversation

@Jithendar11

Copy link
Copy Markdown
Contributor

No description provided.

@Jithendar11

Copy link
Copy Markdown
Contributor Author

Hi @MiloradCvjetkovic ,

Addressing the comments from the previous PR, as referenced in the linked comment

  • Removed the unused cxx_stubs.cpp file.
  • Removed the performance monitor files, as they are no longer used. Please let me know if these files are currently used elsewhere
  • Added the default comment to RockPaperScissors\AppKit-E8_USB\Board\AppKit-E8_M55_HP\RTE\Device\AE822FA0E5597BS0_M55_HP\RTE_Device.h.
  • Regarding the sample SDS_Recordings, no updates should be required since the ML model input and output formats remain unchanged. Please let me know if there are any specific concerns or scenarios that should be covered
  • The CIL static library is used for ML preprocessing and post-processing. This library is generated by Fusion Studio based on the model selected by the user.

@Jithendar11 Jithendar11 marked this pull request as ready for review July 7, 2026 05:02
@MiloradCvjetkovic

Copy link
Copy Markdown
Contributor

Hi @Jithendar11 ,

@Jithendar11

Copy link
Copy Markdown
Contributor Author

@MiloradCvjetkovic
When ALGO_DATA_OUT_BLOCK_SIZE is configured as 4U * sizeof(float), the output SDS files are not flushed in real time, even when --write-flush-records 1 is passed to the sdsio-server Python script. To ensure timely flushing of the output SDS files, the block size has been increased to a larger value. We are extracting the ML input and output from the live-generated .sds files and streaming them to Fusion Studio.

@MiloradCvjetkovic MiloradCvjetkovic self-requested a review July 7, 2026 07:49

@MiloradCvjetkovic MiloradCvjetkovic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@MiloradCvjetkovic MiloradCvjetkovic merged commit c59dd37 into Arm-Examples:main Jul 7, 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.

2 participants