Skip to content

Removed scikit-timeseries migration docs from FAQ #10423

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

Merged
merged 1 commit into from
Jun 24, 2015
Merged

Removed scikit-timeseries migration docs from FAQ #10423

merged 1 commit into from
Jun 24, 2015

Conversation

ihoegen
Copy link
Contributor

@ihoegen ihoegen commented Jun 24, 2015

Issue #10281

@shoyer
Copy link
Member

shoyer commented Jun 24, 2015

LGTM, thanks!

@jorisvandenbossche
Copy link
Member

Indeed, looks good!
Only, I think, when we merge this, the scikit-timeseries website should ideally also be updated: http://pytseries.sourceforge.net/ (it still mentions it is the 'plan' to port it to pandas)

Because it is still downloaded quite a lot: only looking at the last of year and from sourceforge, it has 2000 downloads (or is it downloaded from travis ci or the like?)

@ihoegen
Copy link
Contributor Author

ihoegen commented Jun 24, 2015

I agree

@jreback
Copy link
Contributor

jreback commented Jun 24, 2015

If people really are interested in migration they will hit the pandas page, and eventually ask on the tracker so I don't think that link is ok

@jorisvandenbossche jorisvandenbossche added this to the 0.17.0 milestone Jun 24, 2015
@jorisvandenbossche
Copy link
Member

Anyway, we ourselves can't change that website. So merging this. Thanks @ihoegen !

jorisvandenbossche added a commit that referenced this pull request Jun 24, 2015
Removed scikit-timeseries migration docs from FAQ
@jorisvandenbossche jorisvandenbossche merged commit 8e02d45 into pandas-dev:master Jun 24, 2015
@jorisvandenbossche
Copy link
Member

@pierregm I don't know if we can contact you this way, but: the warning notice on the http://pytseries.sourceforge.net/ website is a little out of date. Would it be possible to update this?

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

Successfully merging this pull request may close these issues.

4 participants