Skip to content
This repository has been archived by the owner. It is now read-only.

Check the variables in the RelayPrioritizer #136

Closed
pastly opened this issue Apr 30, 2018 · 2 comments
Closed

Check the variables in the RelayPrioritizer #136

pastly opened this issue Apr 30, 2018 · 2 comments

Comments

Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
@pastly
Copy link
Member

@pastly pastly commented Apr 30, 2018

During the switch to HTTPS, (the current plan is that) we're going to switch to picking a first hop as usual with the RelayPrioritizer, but then want to pick an exit separately using the first hop's bandwidth as a factor. We will no longer be measuring one relay at a time, and we will be getting results two at a time.

We'll be getting results for relays that the relay prioritizer hasn't returned (yet). And it won't be able to update it's idea of relay priority until it has worked all the way through its current list of relays to return.

This means we might want to make it recalculate best priority more often.

@pastly pastly added this to the Switch to HTTPS milestone Apr 30, 2018
@pastly
Copy link
Member Author

@pastly pastly commented May 9, 2018

I'm currently running with settings that essentially make it return 100 relays at a time.

It seems to take about 2 minutes to do its thing. I should check how often it has to calculate priority again when only returning 100 relays.

@pastly pastly removed this from the Switch to HTTPS milestone Jun 18, 2018
@pastly pastly added this to the 1.0 (MVP nice) milestone Jun 18, 2018
@juga0
Copy link
Contributor

@juga0 juga0 commented Mar 10, 2019

This is not the case anymore, every prioritizer loop returns around 300 relays, and we still consider one result at a time, since results are build without checking the fingerprint in the circuits.
Closed in favor of #29719 and #29720.This is not the case anymore, every prioritizer loop is

@juga0 juga0 closed this Mar 10, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.