Skip to content
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

Message "'0' is not included in Application Times." #1289

Open
PavelBal opened this issue Jan 27, 2025 · 1 comment
Open

Message "'0' is not included in Application Times." #1289

PavelBal opened this issue Jan 27, 2025 · 1 comment

Comments

@PavelBal
Copy link
Member

When running this qualification workflow: https://github.com/mgsc1020/Probenecid-Furosemide-DDI/tree/main I get warnings

Loading project 'Probenecid-Furosemide-DDI', and simulation 'iv, 40 mg (3 min)_Smith 1980_2002+4002_with probenecid.'
27/01/2025 - 14:41:29
! Warning	In createQualificationReport:
'0' is not included in Application Times.

I don't really know what to do with it. Maybe the message can be improved, offering some solutions to the user?

@Yuri05 Yuri05 added this to the 2.3: Qualification workflow milestone Jan 28, 2025
@Yuri05
Copy link
Member

Yuri05 commented Jan 28, 2025

The warning is based on the discussion in #1232
The idea was, that if there is a time offset given, it usually only makes sense if there is an administration starting at the time point of the time offset.

But for the time offset = 0 the warning does not make much sense indeed.
@pchelle Let's remove the check in case TimeOffset=0

checkIsIncluded(timeOffset, allApplicationTimes %||% 0, groupName = "Application Times")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants