Skip to content
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.

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

[docs] Example in the front README page updated #399

Merged
merged 2 commits into from
Apr 30, 2024

Conversation

jjfumero
Copy link
Member

Description

Example in the README page updated.

Problem description

n/ a.

Backend/s tested

Not applicable

  • OpenCL
  • PTX
  • SPIRV

OS tested

Not applicable

  • Linux
  • OSx
  • Windows

Did you check on FPGAs?

If it is applicable, check your changes on FPGAs.

  • Yes
  • No

How to test the new patch?

Not applicable

@jjfumero jjfumero added the documentation Documentation label Apr 30, 2024
@jjfumero jjfumero self-assigned this Apr 30, 2024
Copy link
Collaborator

@mairooni mairooni left a comment

Choose a reason for hiding this comment

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

Just a minor typo but other than that it looks good to me

README.md Outdated
@@ -126,28 +126,37 @@ public class Compute {
}

public void run(Matrix2DFloat A, Matrix2DFloat B, Matrix2DFloat C, final int size) {
TaskGraph taskGraph = new TaskGraph("s0")

// Create a task-graph with multiple tasks. Each tasks points to an exising Java method
Copy link
Collaborator

@mairooni mairooni Apr 30, 2024

Choose a reason for hiding this comment

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

Each tasks -> task

Copy link
Collaborator

@stratika stratika left a comment

Choose a reason for hiding this comment

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

Let's fix the typo and then merge.

@jjfumero jjfumero merged commit fcdebe5 into beehive-lab:develop Apr 30, 2024
2 checks passed
@jjfumero jjfumero deleted the docs/update branch April 30, 2024 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation
Projects
Development

Successfully merging this pull request may close these issues.

3 participants