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

Simplifying Edge AI documentation part. #570

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

Himanshu-kiran
Copy link

  • Please check if the PR fulfills these requirements
  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    This PR introduces some changes in core concepts part of doc understanding the use of Sedna and Ianvs clearly even for new developers.

  • Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
    No

  • Other information:
    Need to simplify sub-part.

Signed-off-by: Himanshu Kiran <[email protected]>
Signed-off-by: Himanshu Kiran <[email protected]>
Signed-off-by: Himanshu Kiran <[email protected]>
Signed-off-by: Himanshu Kiran <[email protected]>
Signed-off-by: Himanshu Kiran <[email protected]>
Signed-off-by: Himanshu Kiran <[email protected]>
Signed-off-by: Himanshu Kiran <[email protected]>
@kubeedge-bot kubeedge-bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 4, 2024
@kubeedge-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
To complete the pull request process, please assign dingyin after the PR has been reviewed.
You can assign the PR to them by writing /assign @dingyin in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubeedge-bot kubeedge-bot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jun 4, 2024
Signed-off-by: Himanshu Kiran <[email protected]>
@Himanshu-kiran Himanshu-kiran changed the title [WIP] Simplifying Edge AI documentation part. Simplifying Edge AI documentation part. Jun 6, 2024
@kubeedge-bot kubeedge-bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label Jun 6, 2024
@Himanshu-kiran
Copy link
Author

Hey @anvithks @edisonxiang Will happy to modify the PR according to your reviews.

@@ -13,9 +13,9 @@ The distributed synergy AI benchmarking Ianvs aims to test the performance of di
The scope of Ianvs includes
- Providing end-to-end benchmark toolkits across devices, edge nodes and cloud nodes based on typical distributed-synergy AI paradigms and applications.
- Tools to manage test environment. For example, it would be necessary to support the CRUD (Create, Read, Update and Delete) actions in test environments. Elements of such test environments include algorithm-wise and system-wise configuration.
- Tools to control test cases. Typical examples include paradigm templates, simulation tools, and hyper-parameter-based assistant tools.
- Tools to control test cases. Typical examples include paradigm templates, simulation tools, and hyper-parameter-based assistant tools. Hyper-parameters are settings that can affect how well your AI models learn from the data.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Choose a reason for hiding this comment

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

The explanation on hyperparameters looks fine to me.

Copy link
Author

@Himanshu-kiran Himanshu-kiran Sep 18, 2024

Choose a reason for hiding this comment

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

@anvithks @edisonxiang Can you look into this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants