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

Suggestion: New structure for the code #9

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

Conversation

cmaureir
Copy link
Collaborator

Hello,
I adapted the README to use Markdown, and also I re-structured the code a bit:

  • I placed all the external files (besides the main) on a separated directory, include.
  • I moved the documentation to a doc directory.

The Makefile was adapted to consider these changes, and also I added a new line to detect the CUDA
installation path.

Cheers

cmaureir and others added 6 commits March 15, 2018 13:20
Signed-off-by: Cristian Maureira-Fredes <[email protected]>
All the objects were being generated on the same
directory as the main, this is not moved to `include/`.

The external CUDA files were also moved to `include/`.

Because of the renaming, the Makefile was adapted to
fit this new structure.

The documentation was also placed under the `doc/`
directory.

Signed-off-by: Cristian Maureira-Fredes <[email protected]>
Signed-off-by: Cristian Maureira-Fredes <[email protected]>
Signed-off-by: Cristian Maureira-Fredes <[email protected]>
@ahwkuepper
Copy link
Owner

@teuben @cmaureir I added both of you as collaborators. I have not received any email notification about your pull requests and only saw this now. Sorry about that. I'm happy that someone is willing to take care of and expand the code. I barely have time to look at it or test new developments.

@ahwkuepper
Copy link
Owner

Thanks for your efforts!

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.

2 participants