Skip to content

LPs on Halide #1814

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
Merged
merged 13 commits into from
Sep 22, 2025
Merged

LPs on Halide #1814

merged 13 commits into from
Sep 22, 2025

Conversation

dawidborycki
Copy link
Contributor

Before submitting a pull request for a new Learning Path, please review Create a Learning Path

  • I have reviewed Create a Learning Path

Please do not include any confidential information in your contribution. This includes confidential microarchitecture details and unannounced product information.

  • I have checked my contribution for confidential information

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the Creative Commons Attribution 4.0 International License.

balintelias
balintelias reviewed Apr 16, 2025
View reviewed changes
stevesuzuki-arm
stevesuzuki-arm reviewed Apr 23, 2025
View reviewed changes
Copy link

@stevesuzuki-arm stevesuzuki-arm left a comment

Choose a reason for hiding this comment

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

I put my 1st round of feedback, only for intro, atm.

stevesuzuki-arm
stevesuzuki-arm reviewed Apr 25, 2025
View reviewed changes
content/learning-paths/mobile-graphics-and-gaming/android_halide/android.md Outdated
## Objective
In this lesson, we’ll learn how to integrate a high-performance Halide image-processing pipeline into an Android application using Kotlin.

## Overview of Mobile Integration with Halide
Copy link

Choose a reason for hiding this comment

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

Can we merge two section "Overview of Mobile Integration with Halide" and "Benefits of Using Halide on Mobile" into one, focusing on how Halide helps some of challenges in image processing on mobile device?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I am leaving it up to Copy Editors

balintelias
balintelias reviewed Jun 30, 2025
View reviewed changes
content/learning-paths/mobile-graphics-and-gaming/android_halide/_index.md Outdated
- Introduction, Background, and Installation.
- Building a Simple Camera/Image Processing Workflow.
- Demonstrating Operation Fusion.
- Integrating Halide into an Android (Kotlin) Project
Copy link

Choose a reason for hiding this comment

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

please make the punctuation consistent between list items.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

?

stevesuzuki-arm
stevesuzuki-arm reviewed Jul 4, 2025
View reviewed changes
Copy link

@stevesuzuki-arm stevesuzuki-arm left a comment

Choose a reason for hiding this comment

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

Please note some of the additional comments are in reply of Resolved comment.

stevesuzuki-arm
stevesuzuki-arm reviewed Jul 4, 2025
View reviewed changes
stevesuzuki-arm
stevesuzuki-arm reviewed Jul 4, 2025
View reviewed changes
Copy link

@stevesuzuki-arm stevesuzuki-arm left a comment

Choose a reason for hiding this comment

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

Please note some comments are in resolved comment

stevesuzuki-arm
stevesuzuki-arm reviewed Jul 4, 2025
View reviewed changes
stevesuzuki-arm
stevesuzuki-arm reviewed Jul 17, 2025
View reviewed changes
Copy link

@stevesuzuki-arm stevesuzuki-arm left a comment

Choose a reason for hiding this comment

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

Note: I will raise a separate review comment for fusion.md

stevesuzuki-arm
stevesuzuki-arm reviewed Jul 17, 2025
View reviewed changes
@pareenaverma pareenaverma merged commit e454013 into ArmDeveloperEcosystem:main Sep 22, 2025
1 check passed
stevesuzuki-arm
stevesuzuki-arm reviewed Sep 25, 2025
View reviewed changes
Copy link

@stevesuzuki-arm stevesuzuki-arm left a comment

Choose a reason for hiding this comment

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

Remaining: processing-workflow.md

stevesuzuki-arm
stevesuzuki-arm reviewed Sep 26, 2025
View reviewed changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pareenaverma pareenaverma pareenaverma left review comments

+2 more reviewers

@stevesuzuki-arm stevesuzuki-arm stevesuzuki-arm left review comments

@balintelias balintelias balintelias left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

@balintelias balintelias

Labels

tech_review

Projects

Arm Learning Paths Roadmap
Status: In Progress

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@dawidborycki @pareenaverma @stevesuzuki-arm @balintelias