Skip to content

Conversation

rundel
Copy link
Contributor

@rundel rundel commented Sep 19, 2025

The underlying issue seems to be that message printing was just looking at show_after value which is fixed. I believe the change below is the desired behavior where we compare now with show_after + last_shown instead.

The code is a bit convoluted because we need to grab last_shown which does not necessarily exist. If it is not yet defined then I've defaulted to -Inf so that the first iteration will always print regardless of show_after.

Fix #668

@rundel
Copy link
Contributor Author

rundel commented Sep 19, 2025

It is also very likely that I am missing necessary PROTECT() and UNPROTECT() calls - it's been way too long since I've worked directly with the C interface.

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.

Never seeing the first progress message from cli_progress_along()
1 participant