Skip to content

Conversation

cragwolfe
Copy link
Contributor

Tweak retry values to be more robust:

Should be equivalent to wait-time-values-between-retries and cumulative-time-between-tries of (in seconds):

>>> a = [min(math.floor(3 * (1.88 ** i)), 720) for i in range(10)]; a; [sum(a[:i+1]) for i in range(len(a))]
[3, 5, 10, 19, 37, 70, 132, 249, 468, 720]
[3, 8, 18, 37, 74, 144, 276, 525, 993, 1713]

Should be equivalent to:

```
>>> a = [min(math.floor(3 * (1.88 ** i)), 720) for i in range(10)]; a; [sum(a[:i+1]) for i in range(len(a))]
[3, 5, 10, 19, 37, 70, 132, 249, 468, 720]
[3, 8, 18, 37, 74, 144, 276, 525, 993, 1713]
```
@awalker4
Copy link
Collaborator

LGTM! If this works well, we can persist in the api spec which will update the values over here.

@awalker4 awalker4 merged commit 974f74b into main Aug 24, 2024
7 checks passed
@awalker4 awalker4 deleted the crag/retry-config-update branch August 24, 2024 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants