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

Change installed name to sknnr #61

Closed
aazuspan opened this issue Oct 4, 2023 · 9 comments · Fixed by #65
Closed

Change installed name to sknnr #61

aazuspan opened this issue Oct 4, 2023 · 9 comments · Fixed by #65
Labels
documentation Improvements or additions to documentation

Comments

@aazuspan
Copy link
Contributor

aazuspan commented Oct 4, 2023

Our previous plan was to use scikit-learn-knn-regression as the installed name of the package and sknnr as the imported name, but per the discussion in #59, we're now tentatively planning to use sknnr across the board.

Pros for switching to sknnr:

  • Shorter, catchier, more memorable
  • No confusion about what name to install and/or import

Cons for switching to sknnr:

  • Less descriptive
  • No explicit reference to scikit-learn

Making the switch will be pretty straightforward, and should just require updating the pyproject.toml, the repository name on Github, and links to the repository throughout the docs (although Github will continue to redirect).

If we do make a final decision to go with sknnr on PyPI, I suggest we make an alpha release just to claim the package name (pre-releases can only be installed with pip install sknnr --pre, so it should be clear that it is not an official, stable release if anyone comes across it).

@aazuspan aazuspan added the documentation Improvements or additions to documentation label Oct 4, 2023
@grovduck
Copy link
Member

grovduck commented Oct 5, 2023

If we do make a final decision to go with sknnr on PyPI, I suggest we make an alpha release just to claim the package name (pre-releases can only be installed with pip install sknnr --pre, so it should be clear that it is not an official, stable release if anyone comes across it).

This will be a brand new step in my evolution (posting a package to PyPI)! Sounds intimidating, but I'll follow your lead here and I'm fully supportive of snagging the name.

@grovduck
Copy link
Member

@aazuspan, wondering if this is the right time to make this package name change now that we have at least three packages that are depending on it (sknnr-spatial, gee-knn-python, and synthetic-knn). I'm still in favor of making this change if you are. I'll probably want to spectate while you create the PyPI entry, though.

@aazuspan
Copy link
Contributor Author

Yeah, great point. I think it's time! I'll rename the repo and make a quick PR with the pyproject.toml updates to get it ready for the alpha release.

I'll probably want to spectate while you create the PyPI entry, though.

Want to do it live over Zoom?

@grovduck
Copy link
Member

Want to do it live over Zoom?

I can't tell if you're mocking me 😉, but, yes, I'd love to see you do that ...

@aazuspan
Copy link
Contributor Author

Haha, not at all! I can screenshare the process, as long as you don't mind watching me fumble my way through the authentication (they just switched how they handle credentials, so my last release took a while to get right!).

@grovduck
Copy link
Member

@aazuspan, I'm @grovduck on PyPI. (You must have changed the repo name yesterday at some point as well?)

@aazuspan
Copy link
Contributor Author

Perfect, just invited you.

You must have changed the repo name yesterday at some point as well?

Yes. Github redirects so it shouldn't be crucial to update anything, but I did update my remote URL with git remote set-url origin https://github.com/lemma-osu/sknnr, just to be on the safe side.

FYI I just found out pip will apparently install a pre-release without the --pre flag if it's the only release available (pypa/pip#5503), so sknnr is installable with just pip install sknnr. We could yank the release if it was a big concern, but I'm not too worried. The upside is we don't need to specify a tag in the downstream package dependencies.

@grovduck
Copy link
Member

I did update my remote URL with git remote set-url origin https://github.com/lemma-osu/sknnr, just to be on the safe side

Ah, thanks for this tip. I would have not done this!

FYI I just found out pip will apparently install a pre-release without the --pre flag if it's the only release available (pypa/pip#5503), so sknnr is installable with just pip install sknnr. We could yank the release if it was a big concern, but I'm not too worried.

Agreed, I'm not too worried either.

@grovduck
Copy link
Member

Just found a small formatting issue on PyPI with footnotes.

pypi
From a quick search of an old SO post, it looks like PyPI now support markdown and is supposed to render exactly as it would on Github:

Currently, PyPI uses cmarkgfm as the markdown renderer, via the readme_renderer library (using readme_renderer.markdown.render(long_description) to produce HTML output). This means that your markdown documents will render exactly the same as on GitHub; it is essentially the same renderer.

It mentions twine here, but I think you said hatch should cover anything that twine does? Happy to open a new issue to track if you think it's worth it.

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

Successfully merging a pull request may close this issue.

2 participants