-
-
Notifications
You must be signed in to change notification settings - Fork 10
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
Stable release channel for TLG catalog #122
Conversation
…lg-catalog into stable-tlg-catalog
…lg-catalog into stable-tlg-catalog
What was the issue with these? |
There were various errors, for example: https://github.com/insightsengineering/tlg-catalog/actions/runs/6297102070/job/17093397369
https://github.com/insightsengineering/tlg-catalog/actions/runs/6296594726/job/17091890939
I think they might be related to some NEST package features used by TLG Catalog being unavailable in package versions released to https://insightsengineering.r-universe.dev/. |
Yeah that makes sense. @insightsengineering/nest-sme - can we get some insight into these errors when you guys get a chance? We can provide more details if needed. |
Very likely that it'll be fixed following the release, yes. We'll just wait till the next release before pushing this out. Thanks, @edelarua! |
Now we have this error when generating the TLG Catalog for stable package versions:
|
@walkowif it seems like the Statistical Engineering team has replaced this function in |
Let's leave this |
Eventually, these two files had to be renamed:
|
Okay we'll revisit these 2 later. Let's get this merged for now |
So now that |
FYI, the MMRM module from the teal.gallery is showing some errors as well. I tried out with different user input values. |
In order to successfully build the TLG Catalog for stable package versions, I had to temporarily remove some contents of files in
book/tables/pharmacokinetic
.Before merging we'll have to figure out what to do about it.
Two TLG Catalog versions (development and stable) deployed concurrently can be viewed here (unless they were overwritten by a scheduled job).
This PR also adds ideas from https://github.com/insightsengineering/idr-tasks/issues/581.