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] Update ROADMAP.md (May/2024). #3382

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

[Docs] Update ROADMAP.md (May/2024). #3382

wants to merge 1 commit into from

Conversation

q82419
Copy link
Collaborator

@q82419 q82419 commented May 2, 2024

No description provided.

Copy link
Member

juntao commented May 2, 2024

Hello, I am a code review bot on flows.network. Here are my reviews of code commits in this PR.


@github-actions github-actions bot added the documentation Improvements or additions to documentation label May 2, 2024
@alabulei1
Copy link
Contributor

I think we should add LLM-related features in the roadmap.

  • Support Stable Diffusion
  • Support burn.rs
  • Support whisper.cpp
  • Support Tensor RT
  • Support Apple MLX
  • Support Intel Intel Extension for Transformers
  • Support llama.c
  • Support running WasmEdge instance on GPU

@alabulei1
Copy link
Contributor

Hi @q82419

Are there any updates on the ROADMA.md? We should also add what we have done since 2024 on the roadmap.md.

@q82419
Copy link
Collaborator Author

q82419 commented Jun 12, 2024

Hi @q82419

Are there any updates on the ROADMA.md? We should also add what we have done since 2024 on the roadmap.md.

If the rule is okey, the above LLM related entries will be added in the next quarter.

@hydai please take a look.

@q82419 q82419 marked this pull request as ready for review June 12, 2024 02:45
@q82419 q82419 requested a review from hydai as a code owner June 12, 2024 02:45
hydai
hydai previously approved these changes Jun 12, 2024
Copy link
Member

@hydai hydai left a comment

Choose a reason for hiding this comment

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

LGTM

docs/ROADMAP.md Outdated
| Proposal | WASM C API proposal | Q4 / 2024 |
| Proposal | WASM memory64 proposal | Q4 / 2024 |
| Feature | DWARF symbol | Q4 / 2024 |
| Proposal | Component model proposal | ?? / 2024 |
Copy link
Member

Choose a reason for hiding this comment

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

Can be Q4 here.

docs/ROADMAP.md Outdated
2. Any individual can request inclusion in the roadmap by commenting on the issue and volunteering as an assignee.
3. WasmEdge maintainers will collect the proposed roadmap entries along with their respective timelines and assignees, then update them within the issue.
4. Existing roadmap entries which not reach their deadlines will be retained in the new quarter.
5. Roadmap entries that have reached their deadlines will be marked as completed or dropped, and the associated issues will be closed.
Copy link
Member

Choose a reason for hiding this comment

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

Complete, drop, or update deadline? Since maybe the entry is delayed by accident.

Copy link
Member

Choose a reason for hiding this comment

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

We should figure out if there is any GitHub-Bot that can help to auto-close the stale proposals.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Sure, I think the deadline can be updated due to some reason. This can be discussed in the new quarter roadmap.

Copy link
Member

Choose a reason for hiding this comment

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

The timeslot is based on quarters now. How about:

  1. Propose a quarter -> Stale
  2. Propose two quarters -> Close
    It should be fine to drop a proposal if we have no plan to implement it in half a year.

Copy link
Member

Choose a reason for hiding this comment

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

Sounds reasonable.

Signed-off-by: YiYing He <yiying@secondstate.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants