Skip to content

DOC: Wrong link in Roadmap: Block Manager Rewrite #51625

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

Closed
1 task done
dshemetov opened this issue Feb 25, 2023 · 11 comments · Fixed by #52396
Closed
1 task done

DOC: Wrong link in Roadmap: Block Manager Rewrite #51625

dshemetov opened this issue Feb 25, 2023 · 11 comments · Fixed by #52396

Comments

@dshemetov
Copy link
Contributor

Pandas version checks

  • I have checked that the issue still exists on the latest versions of the docs on main here

Location of the documentation

https://pandas.pydata.org/docs/development/roadmap.html#block-manager-rewrite

Documentation problem

The link to the design document points to a site by a completely different organization.

Suggested fix for documentation

I don't know where the actual document is or I would make a PR.

@dshemetov dshemetov added Docs Needs Triage Issue that has not been reviewed by a pandas team member labels Feb 25, 2023
@phofl
Copy link
Member

phofl commented Feb 25, 2023

Thx for reporting this.

cc @jorisvandenbossche any idea where the actual documents are?

@datapythonista
Copy link
Member

@datapythonista datapythonista added good first issue Web pandas website Docs and removed Docs Needs Triage Issue that has not been reviewed by a pandas team member Web pandas website labels Feb 27, 2023
@zzmarks26
Copy link

take

@ApolloEagle
Copy link

ApolloEagle commented Feb 28, 2023

@zzmarks26 I think this may be helpful:

Pandas2 Internal Architecture

@zzmarks26
Copy link

@ApolloEagle @datapythonista Just to confirm this is not intended to be rendered right? The link should just point to
https://github.com/wesm/pandas2/blob/master/source/internal-architecture.rst#removal-of-blockmanager--new-dataframe-internals

Thanks!

@datapythonista
Copy link
Member

@ApolloEagle @datapythonista Just to confirm this is not intended to be rendered right? The link should just point to https://github.com/wesm/pandas2/blob/master/source/internal-architecture.rst#removal-of-blockmanager--new-dataframe-internals

It is actually rendered here: https://wesm.github.io/pandas2/internal-architecture.html#removal-of-blockmanager-new-dataframe-internals

But probably no big difference pointing to one or the other.

What I realized, is that we've still got the roadmap duplicated both in the website and in the docs. If you want to fix it in both, that would be great. Thanks!

@ApolloEagle
Copy link

@datapythonista Oops I just saw that you posted that link earlier in the comments. My apologies. Not trying to steal your thunder 😊

@srkds
Copy link
Contributor

srkds commented Apr 3, 2023

Hi @zzmarks26

Are you still working on this issue?
I would like to contribute here.

@zzmarks26 zzmarks26 removed their assignment Apr 3, 2023
@zzmarks26
Copy link

@srkds No I did not get a chance! Go ahead and feel free to assign yourself! Thanks!

@srkds
Copy link
Contributor

srkds commented Apr 4, 2023

take

@srkds
Copy link
Contributor

srkds commented Apr 4, 2023

@srkds No I did not get a chance! Go ahead and feel free to assign yourself! Thanks!

Thanks 😀

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

Successfully merging a pull request may close this issue.

6 participants