46 analysis tool visuals #54
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I fixed some small problems with your PR, you probably mistakenly had a submodule to the tts-be, the constants weren't declared using the regular Python way (all caps at the beginning of the file), you didn't include a valid example for the statistics, and I just added a folder so that the graphs would be saved in a specific place. Further from that the script worked as accordingly, gj! |
SergioEstevao11
previously approved these changes
Mar 9, 2023
SergioEstevao11
approved these changes
Mar 9, 2023
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why was this necessary
To create a more effective and appealing visualization of the JSON output generated by the different courses usage of TTS tool.
How was it done
The statistics api is fetched and kept in variable
stats, then matplotlib functions are used in the created python functionpie_chart,bar_chart,line_chart, to create visually appealing and readable charts that represent the key (course_name):value(usage) pair effectively.After being generated, the chart figures are saved in the
./django/universitydirectory. PS: the images reference were added to.gitignore.How was it tested
This was tested in two ways, the 1st one by retrieving the stats information directly from the endpoint
https://ni.fe.up.pt/tts/api/statistics/and the 2nd by way of previously generated JSON files during the beginning of the UP’s Second Semester (usually one of the peak seasons in terms of TTS usage). In both of those cases the program ran successfullyand the charts were well built.