Skip to content
This repository was archived by the owner on Oct 29, 2024. It is now read-only.

add extra tutorial for timeseries #60

Merged
merged 3 commits into from
Oct 22, 2014
Merged

Conversation

areski
Copy link
Contributor

@areski areski commented Oct 16, 2014

No description provided.

@jvshahid
Copy link
Contributor

Can you choose a better name for the new example. example_timeseries isn't descriptive enough, besides the other example demonstrates writing and reading timeseries.

@areski
Copy link
Contributor Author

areski commented Oct 16, 2014

What about 'server_data' ?

@jvshahid
Copy link
Contributor

I was thinking about data_aggregation, really what's extra in the new example is that you insert more data and run a complicated query that has aggregation, isn't that right ?

@areski
Copy link
Contributor Author

areski commented Oct 16, 2014

In the file tutorial.py, the example just create few points and doesn't really show the time series or how a full usage will be, I liked the idea to have more extended example.

The new example create a serie "server_data.cpu_idle" with the following columns "time", "value", "hostName" then it generate a bunch of fake data in it for a configurable number of days.

What about 'tutorial_server_data.py' ?

@areski
Copy link
Contributor Author

areski commented Oct 16, 2014

maybe it will be nice to have an other example for sine wave as you have here:
https://github.com/influxdb/influxdb-ruby

The main goals is to have programmers having further examples to play with and get the glance how to use the influxDB from python.

@jvshahid
Copy link
Contributor

I'm fine with tutorial_server_data and the sine wave example

@areski
Copy link
Contributor Author

areski commented Oct 17, 2014

@jvshahid it's now all there with the last commit.

jvshahid added a commit that referenced this pull request Oct 22, 2014
add extra tutorial for timeseries
@jvshahid jvshahid merged commit 315e6c0 into influxdata:master Oct 22, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants