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

Update skills.md #1038

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

Update skills.md #1038

wants to merge 1 commit into from

Conversation

wsphd
Copy link

@wsphd wsphd commented Jan 5, 2023

Added FPGA to GPU Programming

Description

Added FPGA to GPU Programming

Motivation and Context

This adds more granularity and detail to hardware-level programming.

Checklist:

  • I have posted the link for the PR in the usrse slack (#website) to ask for reviewers
  • I have previewed changes locally
  • I have updated the README.md if necessary

cc @usrse-maintainers

Added FPGA to GPU Programming
@jdamerow
Copy link
Contributor

jdamerow commented Jan 5, 2023

I'm not an exert on this, but should it be its own point or are those two things (GPU and FPGA) typically listed together?

@cosden
Copy link
Member

cosden commented Jan 5, 2023

Thanks for the addition @wsphd! We clearly missed FPGAs. I'd vote to list FPGA as a separate item. The small number of people I know that do FPGA work, also do GPU (and are quite knowledgable in both), but the reverse isn't necessarily true. It seems unique enough we could list it separately. And we should list it, it's clearly a growing area.

@JeffCarver
Copy link
Contributor

I am good with listing FPGA as a separate item.

@wsphd
Copy link
Author

wsphd commented Jan 5, 2023 via email

@namurphy
Copy link
Contributor

namurphy commented Jan 5, 2023

Thank you for the contribution! I think adding FGPA as a separate item would be worthwhile. Since FGPA is less familiar as an acronym than GPU, would it be helpful to spell out the full name?

I've got several more additions as well. What's the favor of the group? Do a separate PR for each or batch them together as one PR?

That sounds great! My inclination would be to batch them together as a single PR.

@wsphd
Copy link
Author

wsphd commented Jan 5, 2023 via email

@namurphy
Copy link
Contributor

namurphy commented Jan 5, 2023

Nick, A compromise might be to just link it to the Wikipedia page (or similar) for the acronym but that approach is true for many other elements as well.

I had actually begun to think the same thing! Having Wikipedia links for less well known items on the list could be potentially quite helpful.

@cosden
Copy link
Member

cosden commented Jan 25, 2023

I really like the suggestion of linking to wikipedia pages! Great idea. @wsphd want to make FPGA a separate entry (I think being verbose is fine!) and add more (either in this PR or in a new one, whatever is easier)?

@kescobo
Copy link
Contributor

kescobo commented Jun 18, 2023

Hi all - this seems to have gone a bit stale. @wsphd would you still like to complete this? It looks like folks decided on

  1. having separate list items
  2. linking to Wikipedia (at least for these new entries)

I think adding other links can be a separate PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants