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

PyWavelets vs. nigma Github repo #90

Closed
cdeil opened this issue Jul 31, 2015 · 13 comments
Closed

PyWavelets vs. nigma Github repo #90

cdeil opened this issue Jul 31, 2015 · 13 comments
Assignees

Comments

@cdeil
Copy link

cdeil commented Jul 31, 2015

I noticed that the "fork on Github" button at http://pywavelets.readthedocs.org/ still points to https://github.com/nigma/pywt

So I wanted to change it to point to https://github.com/PyWavelets/pywt and use Github search to find the location in the code / docs where this is set.

But code search in forked repos doesn't work: https://help.github.com/articles/searching-code/#search-by-the-number-of-forks-the-parent-repository-has

I'm not sure ... maybe this could be resolved by deleting https://github.com/nigma/pywt and then forking it from https://github.com/PyWavelets/pywt ?
https://help.github.com/articles/what-happens-to-forks-when-a-repository-is-deleted-or-changes-visibility/

cc @nigma @rgommers

@jenshnielsen
Copy link

I have solved a number of similar issues by contacting github support, explain the situation and ask them to detach the repository from the original one.

@aaren
Copy link
Member

aaren commented Jul 31, 2015

@cdeil good spot, thank you. Agreed, it's annoying that you can't search in forks!

@cdeil
Copy link
Author

cdeil commented Jul 31, 2015

This issue was closed, but is only partly adressed.
I think what @jenshnielsen proposed, that you ask Github to detach this fork (or revert which is the fork) would be a good solution.

@aaren
Copy link
Member

aaren commented Jul 31, 2015

@cdeil sorry, I was overeager. I think I understand now - the problem is that the code in this repository is unsearchable in the browser (the links were a separate problem), right?

This is caused by this repo being a fork of nigma/pywt.

@aaren aaren reopened this Jul 31, 2015
@cdeil
Copy link
Author

cdeil commented Jul 31, 2015

@aaren – Yes.

@aaren
Copy link
Member

aaren commented Aug 3, 2015

I've contacted github support.

@aaren
Copy link
Member

aaren commented Aug 3, 2015

Ok, Pywavelets/pywt added to index and code is now searchable.

@aaren aaren closed this as completed Aug 3, 2015
@aaren
Copy link
Member

aaren commented Aug 3, 2015

Something else we could do: make PyWavelets the root repository so that future forks show up as forks of PyWavelets/pywt rather than nigma/pywt.

@rgommers
Copy link
Member

rgommers commented Aug 3, 2015

Nice quick resolution of this issue.

Something else we could do: make PyWavelets the root repository so that future forks show up as forks of PyWavelets/pywt rather than nigma/pywt.

I think forks that are made from PyWavelets/pywt will anyway show up as forked from there, see https://github.com/PyWavelets/pywt/network and https://github.com/aaren/pywt for example. So I don't think it's a major issue that this repo says it's forked from @nigma's repo - because that is the actual history of the project.

@cdeil
Copy link
Author

cdeil commented Aug 3, 2015

It's still confusing to have the active repo shown as a fork.
Also https://github.com/nigma/pywt currently has 133 stars, but https://github.com/PyWavelets/pywt only 23, so I think Google (and Github?) search will point people there.

One other thing you could do is close the issue tracker on the old repo: https://github.com/nigma/pywt/issues
And close the PRs on the old repo (https://github.com/nigma/pywt/pulls), leaving a comment pointing to the new repo.

@rgommers
Copy link
Member

rgommers commented Aug 3, 2015

Good points. I can update https://github.com/nigma/pywt, so its README points here and I can close the issue tracker (assuming Github lets me do that without deleting the issue history).

aaren added a commit to aaren/pywt that referenced this issue Aug 11, 2015
@rgommers
Copy link
Member

@nigma had already updated the README on https://github.com/nigma/pywt, it now points prominently to this repo

@rgommers
Copy link
Member

All PRs on https://github.com/nigma/pywt are closed, and I've only left a single issue open (nigma/pywt#13) which is titled "Continuing PyWavelets development on PyPi" and still needs resolving.

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

4 participants