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

Fix bug of different-scale option about indentation #49

Merged
merged 1 commit into from
Nov 19, 2019

Conversation

HiromuIshikawa
Copy link
Contributor

This PR purposed to fix bug of --different-scale option caused by wrong indentation.

Used data: data/ex8.dat

# Example Data Set 8 with Categories of different scale
@ People,Income
2007,100,500000
2008,450.23,760000
2009,255,1000000
2010,340,800000
2011,160,600400
2012,200,950000
2014,488,700000

When I plotted 2 categories data (that is above one) having different-scale, the output is as follows:
スクリーンショット 2019-11-18 16 08 12

However, I think the ideal output is to have two normalized plots by category.
So, when I was reading source code of termgraph, I found a wrong indentation in block treating different-scale.

The output after fixing indentation is as follows:
スクリーンショット 2019-11-18 16 15 47

@mkaz mkaz merged commit 5722c0e into mkaz:master Nov 19, 2019
@mkaz
Copy link
Owner

mkaz commented Nov 19, 2019

Nice fix 👍
Thank you for the contribution

@HiromuIshikawa HiromuIshikawa deleted the fix_different-scale branch November 19, 2019 14:05
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

Successfully merging this pull request may close these issues.

2 participants