-
Notifications
You must be signed in to change notification settings - Fork 163
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
Add UV mirrors settings #2333
Add UV mirrors settings #2333
Conversation
ffb1fd4
to
d68f6a7
Compare
Updated dependencies detected. Learn more about Socket for GitHub ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Current mechanism looks good. Additional thoughts:
-
On the "bad connection" page, should we first add a banner warning that says "Bad access to python mirror. The download might fail", similar to the bottom banners on the "good" or "loading" pages?
-
On this “bad connection” page, users might first try to switch their network before setting mirrors. Do we have a refresh mechanism that enables the page to re-evaluate? (I'm sorry I can't directly check from the code)
Looks good! Will we provide a default mirror for python source? |
const checkPypiMirrorReachable = async (mirror: string) => { | ||
return ( | ||
isValidUrl(mirror) && | ||
(await electronAPI().NetWork.canAccessUrl( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this is for pypy
which is different from pypi.
https://docs.astral.sh/uv/configuration/environment/#uv_pypy_install_mirror
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes we need to fix this. We probably also want to verify connection to torch mirror.
See |
Fixed in #2339. User can refresh by clicking the status icon, i.e. the red cross. |
7759c99
to
5616a0a
Compare
Requires:
This PR adds python/pypi mirrors configuration to the install wizard. Users will be prompted to configure mirrors if they don't have access to the default python mirrors.
Good connection
Bad connection
Loading state
┆Issue is synchronized with this Notion page by Unito