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.

Sign up for GitHub

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

Jump to bottom

Change graph export format from PDF to PNG and remove axis scaling #103

Merged
merged 1 commit into from
Nov 2, 2018
Merged

Change graph export format from PDF to PNG and remove axis scaling #103

merged 1 commit into from
Nov 2, 2018

Conversation

JasbirShah
Copy link
Collaborator

@JasbirShah JasbirShah commented Nov 1, 2018 •
edited
Loading

Changes

  • [ x ] Export graphs as PNGs instead of PDFs to vastly reduce export times.
  • [ x ] Disable logarithmic scaling of the axes of the solution fit plot.

@JasbirShah
Change graph export format from PDF to PNG and remove axis scaling
f4df7a6 
Refactor: graphs are exported as PNGs instead of PDFs vastly reducing export times.
Refactor: stops logarithmically scaling the axes of the solution fit plot.
@JasbirShah JasbirShah self-assigned this Nov 1, 2018
@JasbirShah JasbirShah added the cli Related to the command line interface label Nov 1, 2018
@jayden626
Copy link
Collaborator

Looks like this adds constants into the CLI? Did this not affect the parser or is it not applied?

@jayden626 jayden626 added this to the 0.7.0 milestone Nov 2, 2018
@jayden626 jayden626 merged commit ab40aea into develop Nov 2, 2018
@ScottWalkerAU ScottWalkerAU deleted the feature/remove-solution-fit-scaling-in-cli branch November 2, 2018 02:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jayden626 jayden626 jayden626 approved these changes

@ScottWalkerAU ScottWalkerAU Awaiting requested review from ScottWalkerAU

Assignees

@JasbirShah JasbirShah

Labels
cli Related to the command line interface
Projects
None yet
Milestone
0.7.0
Development

Successfully merging this pull request may close these issues.
2 participants
@JasbirShah @jayden626