-
Notifications
You must be signed in to change notification settings - Fork 191
HTML-Heatmap-Generation in futhark profile
#2340
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
Merged
+1,000
−64
Merged
Changes from 42 commits
Commits
Show all changes
63 commits
Select commit
Hold shift + click to select a range
75f5ee9
Scaffolding and Setup for HTML-Generation
Hyalunar 90b7e4e
Add the htmlDir Parameter to Record creation
Hyalunar cf9a88e
SourceRange Parser, Event Splitting
Hyalunar 254babb
Enforce invariants in SourceRanges
Hyalunar bccfe03
Improve Variable Naming
Hyalunar 8f843fd
Refactor the Provenance Parsing
Hyalunar b0e48ba
Load all source files
Hyalunar b99adb8
Group Summaries By File, Don't distribute event time
Hyalunar 8954373
WIP: Find all Overlapping Ranges
Hyalunar a6a96ec
Simplified Range overlap check
Hyalunar 90c5606
WIP: SourceRange merging
Hyalunar 9f864f6
Handling of the first two cases
Hyalunar 9b6d8e0
Simplification of `mergeRanges`
Hyalunar 78edfef
Filter Empty SourceRanges after merging
Hyalunar 45d06f7
Finish `summarizeAndSplitByFile`
Hyalunar 76be8f4
Fix `sourceRangeOverlapsWith` to respect exclusiveness
Hyalunar 38a724f
Start: HTML-File-Generation
Hyalunar e908946
Refactor SourceRange into a Single Module
Hyalunar 21c1ba2
Break out `EventSummary` to avoid circular imports
Hyalunar 88f1da9
I forgot to run the formatter
Hyalunar f34f913
Split out headHtml and relativise for futher usage
Hyalunar d098fc3
Write out the empty html files
Hyalunar 9b6f070
Prettify the file paths
Hyalunar 15459d7
Generate HTML with Spans for Cost-Centres
Hyalunar 1359e6f
Simple Source Highlighting
Hyalunar 46870eb
Normalize CRLF and CR newlines
Hyalunar b0eeb23
Add hover information
Hyalunar 87639c5
Embed CSS file, add anchors
Hyalunar 2f82eba
Prepare Anchors for Details
Hyalunar dcf27ee
Display Detail Tables for Source Ranges
Hyalunar 97049e4
Extract html generation from file writing
Hyalunar 488a0e1
Cost Centre Listing at `cost-centres.html`
Hyalunar ae4b8b3
Prepare Code Cleanup and Organisation
Hyalunar c948ab0
Rewrite `ccOverview` to new data structures
Hyalunar 94df9e0
Rewrite `heatmapHtml` to new details structure
Hyalunar 4ef8553
Type Aliases
Hyalunar 40b28e0
Code Rewrite To Organized Data Structures
Hyalunar e5174be
Formatting and Hyperlinking
Hyalunar 3710ae7
Fixed the inverted heatmap colors
Hyalunar 579b57b
Fix build.
athas 52f75a4
Browser-Readable Profiling Information
Hyalunar 3903cf9
document the newly generated files
Hyalunar 35c57ff
improve the `Files produced` docs
Hyalunar 59b9b5c
Remove redudant import; removes the warning, fixes CI
Hyalunar 06ce1f7
Don't use relative paths to write files
Hyalunar 33296be
Add futhark-profile/style.css to extra-source-files.
athas f432a80
Style fix.
athas 1704c12
Create the Directory even if there are no source files
Hyalunar 9eab85d
Unify CSS.
athas 4359b06
Style fix.
athas b3b373c
Fix to the `Map.!` issue, unformatted and untested
Hyalunar 05e69a2
Postponed formatting
Hyalunar a17a40a
Document `foo-index.html` to be in the toplevel-directory
Hyalunar c3a2f5b
Style fix.
athas 9c544d9
Merge branch 'master' into profile-html-heatmap
athas b46d496
Scaffolding for HTML Index file
Hyalunar 438db43
Correctly Link the Css File
Hyalunar de1f6c1
Index File With Explanations
Hyalunar d02583e
Fix Heatmap Title and apply hlint functor law
Hyalunar f94d33f
Remove a trailing whitespace
Hyalunar 0b03fd6
Changelog: HTML-Generation in `futhark profile`
Hyalunar 928bceb
Final `ormolu` formatting
Hyalunar fc6595b
Augment manpage slightly more.
athas File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,15 @@ | ||
| a.silent-anchor { | ||
| text-decoration: inherit; | ||
| color: inherit; | ||
| } | ||
|
|
||
| table, th, td { | ||
| border: 1px solid black; | ||
| border-collapse: collapse; | ||
| } | ||
|
|
||
| th, td { | ||
| padding-left: 0.5em; | ||
| padding-right: 0.5em; | ||
| text-align: center; | ||
| } |
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would prefer if this was put immediately in the top level directory, for convenience. I have half a plan to make all the profiling information available in HTML format eventually.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can do that, I chose it this way because
index.htmlin a directory are sometimes conveniently implicitly rendered as a directory listing. Usingfoo-index.htmlwould not accomplish that.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Source File Renderings must be in a nested directory, otherwise there would be conflicts between different benchmarks.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, I don't want all HTML files to be in the root, only a single
index.html.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The documentation is changed since a17a40a but it's not implemented yet.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Implemented in de1f6c1, is this what you expected?