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

feature request : PNG file from current graph #129

Closed
YouriAndropov opened this issue Mar 11, 2015 · 3 comments
Closed

feature request : PNG file from current graph #129

YouriAndropov opened this issue Mar 11, 2015 · 3 comments

Comments

@YouriAndropov
Copy link

Hi,

I would be nice if we could get a PNG of the current graph.
As you can open more than one gc log simultaneously, this feature should support that also.
We could keep the graph current height and width, or use particular values, eventually.

Thank you.

As a bonus, we could also be able to set the PNG height and width from the command line, and add or remove some metrics (permgen, young gen usage, old gen usage, ...)

@aolle
Copy link

aolle commented May 1, 2015

Hi,

I forked the project. I'll implement this feature and I'll send a pull request. It's really nice when we need to get the graphs when we're making infraestructure reports at work.

Thank you.
Àngel Ollé Blázquez.

chewiebug added a commit that referenced this issue May 3, 2015
@chewiebug chewiebug added this to the 1.35 milestone May 3, 2015
@chewiebug chewiebug assigned chewiebug and unassigned chewiebug May 3, 2015
@chewiebug
Copy link
Owner

Hi Àngel,

Thank you very much for your contribution, I have merged #131.

I leave the issue open, because the "bonus" mentioned is not implemented (export chart with current settings).

Best regards,
Jörg

chewiebug added a commit that referenced this issue May 13, 2015
* develop:
  reduce number of WARNING messages when parsing introduction of a detailed G1 message fails (#128)
  add Àngel Ollé Blázquez to list of contributors
  fix merge problem between #129 and #130
  export graph to PNG fature using the GCViewer GUI
  Command line graph generation random issue
  parallel scavenge collector: handle combination of -XX:+PrintAdaptiveSizePolicy, -XX:+PrintTenuringDistribution, -XX:+PrintApplicationStoppedTime (#128)
  fix negative pause for VM_OPERATION event near parser problem (#128)

Conflicts:
	src/main/java/com/tagtraum/perf/gcviewer/exp/impl/DataWriterFactory.java
	src/main/java/com/tagtraum/perf/gcviewer/view/SimpleChartRenderer.java
	src/test/java/com/tagtraum/perf/gcviewer/imp/TestDataReaderSun1_7_0G1.java
	src/test/java/com/tagtraum/perf/gcviewer/imp/TestDataReaderSun1_8_0G1.java
chewiebug added a commit that referenced this issue May 13, 2015
* develop:
  reduce number of WARNING messages when parsing introduction of a detailed G1 message fails (#128)
  add Àngel Ollé Blázquez to list of contributors
  fix merge problem between #129 and #130
  export graph to PNG fature using the GCViewer GUI
  Command line graph generation random issue
  parallel scavenge collector: handle combination of -XX:+PrintAdaptiveSizePolicy, -XX:+PrintTenuringDistribution, -XX:+PrintApplicationStoppedTime (#128)
  fix negative pause for VM_OPERATION event near parser problem (#128)
areese pushed a commit to areese/GCViewer that referenced this issue Oct 5, 2015
@chewiebug chewiebug removed this from the 1.35 milestone Aug 1, 2018
chewiebug added a commit that referenced this issue Sep 14, 2019
…-use-current-configuration

Feature/#129/fix png export to use current configuration
@chewiebug
Copy link
Owner

#227 adds exporting of the png file using the current settings -> closing this issue.

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

No branches or pull requests

3 participants