jbush001 / NyuziProcessor

GPGPU microprocessor architecture
Apache License 2.0
1.97k stars 349 forks source link

Issue #57 resolved, multiplication underflow #116

Closed manili closed 6 years ago

manili commented 6 years ago

Sorry about that. I use the "New pull request" button on top of your repository. I can not understand why Github also include my previous changes and also I cannot exclude them from this pull request!

jbush001 commented 6 years ago

I think you need to integrate changes from my repository into your fork. This is an important thing to know how to do. There are instructions here: https://help.github.com/articles/syncing-a-fork/

But I'll summarize:

Do this only once:

$ git remote add upstream https://github.com/jbush001/NyuziProcessor

Do this any time you want to sync your branch:

$ git fetch upstream
$ git merge upstream/master

Once you've done this, you should be able to create a new pull request.

manili commented 6 years ago

Sorry for the newbie's mistakes. It looks like that everything's OK, because now GitHub tell me we have identical master branches. I'll do my best next time.

jbush001 commented 6 years ago

No need to apologize, you're doing this for the first time and that's how you learn. Unfortunately, some of this stuff is not very obvious.