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

Mind map #2721

Closed
wants to merge 6 commits into from
Closed

Mind map #2721

wants to merge 6 commits into from

Conversation

shoshijak
Copy link
Contributor

Mind map of certain parts of the HPX source code, the API, hierarchy of classes etc. that I made during my internship at CSCS (March - June 2017) and could be useful for other people new to HPX (interns, students, GSoC, etc.).

- complete Readme sheet
- update the class hierarchy to reflect the changes introduced by
the resource partitioner
- clean up the pop-up notes
@preejackie
Copy link

@shoshijak Thank you very much. It will be a huge help for beginners!

@hkaiser
Copy link
Member

hkaiser commented Jun 28, 2017

@shoshijak I assume this is generated somehow using some tools. Wouldn't it be better if we added a build step to the documentation tool chain which generates this on demand and uploads this as part of the documentation (http://stellar-group.github.io/hpx/docs/html/)? Otherwise the mind-map will become outdated very quickly...

@hkaiser
Copy link
Member

hkaiser commented Jun 28, 2017

@shoshijak: how did you generate this?

@mcopik
Copy link
Contributor

mcopik commented Jun 29, 2017

@shoshijak I don't think that adding hundreds of images to the repository is a good idea. It might be generated during build but it should not be shipped with the source code.

@hkaiser
Copy link
Member

hkaiser commented Jul 20, 2017

Closing this as this needs build system integration and automation instead of dropping in a snapshot of things.

@hkaiser hkaiser closed this Jul 20, 2017
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.

None yet

4 participants