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

Mirko bachelor thesis #637

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

Mirko bachelor thesis #637

wants to merge 12 commits into from

Conversation

Migarl
Copy link
Collaborator

@Migarl Migarl commented Aug 22, 2023

Add profiler report parser for gprof, perf and hprof.
I also copied the parser for JProfiler from one participant of my study. Debugging is not working yet for this profiler...

@falko17
Copy link
Collaborator

falko17 commented Aug 24, 2023

Before I review this: Wouldn't it make sense to move the profiling2gxl tool into its own repository? It seems to be independent of SEE (and has its own README, gitignore, etc), so I think it would make sense to maintain it separately. If it would still be useful to have it within the SEE repository, we could include it as a git submodule.

@Migarl
Copy link
Collaborator Author

Migarl commented Aug 25, 2023

@koschke What do you say?

@Migarl Migarl added the bachelorproject Something applicable to the bachelor project, e.g. its report label Aug 25, 2023
@koschke
Copy link
Collaborator

koschke commented Aug 25, 2023

@koschke What do you say?

There is an overlap with our GXLWriter. Currently it has been reused by copy and paste. For this reason it might make sense to keep it in the same repository. In the long run we should remove the redundancy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bachelorproject Something applicable to the bachelor project, e.g. its report
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants