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

[KIEKER-1141] Rework kieker.tools.traceAnalysis.filter.visualization #2783

Open
rju opened this issue Nov 12, 2024 · 1 comment
Open

[KIEKER-1141] Rework kieker.tools.traceAnalysis.filter.visualization #2783

rju opened this issue Nov 12, 2024 · 1 comment

Comments

@rju
Copy link
Member

rju commented Nov 12, 2024

JIRA Issue: KIEKER-1141 Rework kieker.tools.traceAnalysis.filter.visualization
Original Reporter: Nils Christian Ehmke


I think a part of this has already been discussed during our meetings.

I suggest to rework the classes within the kieker.tools.traceAnalysis.filter.visualization package.
1. Some of the filters use the graph from the graph package. Some, however, do not. We should consider to improve some of the older filters and maybe even rely fully on the graph (Maybe we could even make the repositories more lightweighted or even remove them). This should of course be planned, as this would change a lot of the filters.
1. The graph class is very abstract. Perhaps even a little bit too abstract. We should simplify the structure.
1. We should extend the graph class in order to support decorators as described in KIEKER-803 Done . I am currently not sure whether this is possible with the current state of the class or not.
1. Some of the filters (e.g., the DescriptionDecoratorFilter) use a lot of generics. I am not sure whether this is a good idea. This should be discussed.
1. Configurable time units in response times ( KIEKER-1200 Done , KIEKER-1059 Done )

@rju
Copy link
Member Author

rju commented Nov 12, 2024

author André van Hoorn -- Mon, 29 Jul 2013 17:30:06 +0200

Possibly reengineer if working on extensions like KIEKER-803 Done .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant