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

PP 1617 - visualisation #231

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

PP 1617 - visualisation #231

wants to merge 50 commits into from

Conversation

Skaduwa
Copy link
Collaborator

@Skaduwa Skaduwa commented Jul 7, 2017

The visualisation approach we had in mind is now implemented.
We would like to finish the Project soon and would be glad to get a review form @kevin-klein.

Documentation can be found in templates/ReadMe.md
and in the ReadMe of the modified modules

Andre Emmerichs and others added 30 commits November 26, 2016 15:03
Created new branch for project practical
Adde first idea of ncloc module, need to add tests & more languages
later
Prototype for comment extactor, added "beta" version to support
development of other modules. Improvements will come in the next few
days, but JSON structure will keep equal over development
Now comments in the same line with code can be found, tests still
missing
Split up tests and program in sperate files, renamed folder name
Added visualisation approach with matplotlib/numpy for module
locPerContribution, visualisation is saved in
data/views/locPerContribution
added module support
added samples for all types
Please see comment in program.py - create_barchart
TODO:
Create template folder in modules/visualization with templates
"chartname".html (see function program.py-copyTemplateToTarget)
Create index.html file, which gets copied to mainPath(see
program.y-assemble_pie_chart) and reads out links.json (-manageLinkFile)
to create links
General improvements of everything
TODO: Need a template folder in 101worker directory and more
added License information.
added license information
Added test cases and ReadMe
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.

2 participants