Skip to content

Coerce all file durations with floor instead of ceil.#3685

Merged
rtibbles merged 1 commit intolearningequality:hotfixesfrom
rtibbles:hunting_for_ceils
Sep 28, 2022
Merged

Coerce all file durations with floor instead of ceil.#3685
rtibbles merged 1 commit intolearningequality:hotfixesfrom
rtibbles:hunting_for_ceils

Conversation

@rtibbles
Copy link
Copy Markdown
Member

@rtibbles rtibbles commented Sep 27, 2022

Summary

Description of the change(s) you made

  • Uses floor instead of ceil for file durations
  • Do this to ensure that duration never exceeds actual duration and makes it tricky to get full progress for duration based progress
  • If ceil-ed, then users might have to skip back a second in a video in Kolibri for it to be marked as complete.

Address concerns raised here: #3317 (comment)

Copy link
Copy Markdown
Member

@bjester bjester left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

Do this to ensure that duration never exceeds actual duration and makes it tricky to get full progress for duration based progress.
@rtibbles rtibbles merged commit 796b011 into learningequality:hotfixes Sep 28, 2022
@rtibbles rtibbles deleted the hunting_for_ceils branch September 28, 2022 00:30
@bjester bjester mentioned this pull request Oct 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants