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

Highcharts licence #2

Closed
miniche opened this issue Jan 14, 2014 · 11 comments
Closed

Highcharts licence #2

miniche opened this issue Jan 14, 2014 · 11 comments
Assignees

Comments

@miniche
Copy link
Member

miniche commented Jan 14, 2014

The Highcharts library is just awesome: user interaction, real time...
Its licence is free for open-source project. I think we may contact the developers to ask them whether CentralReport (host part) Is in agreement with the licence. In fact, CentralReport can be used by companies and, in this case, we are "out" of the licence.

We can consider other open-source libraries, like Flot.

What do you think about the following ones:

@ghost ghost assigned Ninir Jan 15, 2014
@Ninir
Copy link
Member

Ninir commented Jan 15, 2014

There is also Chart.js, made with canvas only. It's been a long time since an update, the author is quite of busy, but the project is promising :)

@Ninir
Copy link
Member

Ninir commented Jan 15, 2014

Let also ask them if they can provide a year-free license :)

@miniche
Copy link
Member Author

miniche commented Jan 15, 2014

I will write to Highcharts to ask them :)

Chart.js is a good library, but as it's build with canvas, it doesn't allow user interaction (http://www.chartjs.org/docs/#generalIssues-chartInteractivity). When you have a long timeline with multiple events, it could be very useful to know a precise value with the mouse cursor, doesn't it?

For pie charts, it isn't really mandatory, but it is for a timeline or bars.

@miniche
Copy link
Member Author

miniche commented Jan 15, 2014

Another one: Morris

@Ninir
Copy link
Member

Ninir commented Jan 15, 2014

@miniche With the default library, it does not, just reading the link you provided. However, It starts to: https://github.com/nnnick/Chart.js/pull/137/files

Chart.js just needs events for hovering/clicking on series points, etc...

@miniche
Copy link
Member Author

miniche commented Jan 15, 2014

@Ninir Oh, excellent!
This pull request has been opened since seven months now... Any chance to be available one day in master you think?

@miniche
Copy link
Member Author

miniche commented Jan 15, 2014

@Ninir Another PR with tooltip support in Chart.js: chartjs/Chart.js#51

@Ninir
Copy link
Member

Ninir commented Jan 15, 2014

The issue here is about the author itself... he's been busy for a few months, but he wants to be on track again asap. Let's see how this evolves, but for now, it's too light for us I guess.

@miniche
Copy link
Member Author

miniche commented Jan 15, 2014

Mmh. Chart.js with the interaction layout may be the better chart library.
We could follow his activity and his refactoring work, and we could consider implementing the next version in CentralReport.

For now, I will write an email to the Highcharts developers to know whether the client part of CR can include their library.
For future projects together, we can also consider including another JS library, with a free licence.

Rickshaw seems to be very good to draw timelines but doesn't support donut or pie charts.
Flot can be a good compromise I think in this case.

@miniche
Copy link
Member Author

miniche commented Jan 15, 2014

I have written to Highcharts about our project, I'm waiting for the reply :)

In the last commit in the "init" branch, I have included the Flot library and removed Highcharts. It's just a detail, we can focus on the main goal of this repository: the design! :D

@miniche
Copy link
Member Author

miniche commented Jan 24, 2014

Solved. Highcharts cannot be free for our project. So... Already replaced by Flot! Go go go!

@miniche miniche closed this as completed Jan 24, 2014
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

2 participants