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

Document addition and improvement #192

Merged
merged 4 commits into from
Sep 11, 2019
Merged

Conversation

marktwtn
Copy link
Collaborator

  • Change the document file mode
  • Improve the document of threading model
  • Add the new document of running remote worker on DE10-Nano

@marktwtn marktwtn self-assigned this Sep 10, 2019
@jserv
Copy link
Member

jserv commented Sep 10, 2019

Change docs/de10-nano.md to docs/board-de10-nano.md

docs/de10-nano.md Outdated Show resolved Hide resolved
Copy link
Member

@ender503 ender503 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@marktwtn
Copy link
Collaborator Author

Change docs/de10-nano.md to docs/board-de10-nano.md

Done.

@jserv
Copy link
Member

jserv commented Sep 10, 2019

Mention ``board-de10-nano.md` as an instance of remote interface.

@marktwtn
Copy link
Collaborator Author

Mention ``board-de10-nano.md` as an instance of remote interface.

I guess this should be mentioned in the docs/remote-interface.md?

@jserv
Copy link
Member

jserv commented Sep 10, 2019

Mention ``board-de10-nano.mdas an instance of remote interface. I guess this should be mentioned in thedocs/remote-interface.md`?

ACK. Clarify the relationship between DE10 Nano board and remote interface.

@marktwtn
Copy link
Collaborator Author

ACK. Clarify the relationship between DE10 Nano board and remote interface.

I added the description of remote worker in remote-interface.md and give a hyperlink to board-de10-nano.md.

@jserv jserv merged commit 7b8a253 into DLTcollab:develop Sep 11, 2019
@marktwtn marktwtn deleted the document branch September 11, 2019 10:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants