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

PetabImporter.create_edatas creates incomplete edata objects. #1466

Open
FFroehlich opened this issue Sep 16, 2024 · 1 comment
Open

PetabImporter.create_edatas creates incomplete edata objects. #1466

FFroehlich opened this issue Sep 16, 2024 · 1 comment

Comments

@FFroehlich
Copy link
Contributor

This function appears pretty useless on it's own as fixed parameters etc from the condition table are not filled in. Would expect that the generated objects have partial information that is set by amici.petab.conditions.fill_in_parameters.

@dweindl
Copy link
Member

dweindl commented Sep 16, 2024

I think that was intentional, but the intention is not sufficiently-well documented.
Nevertheless, I agree that it's probably less surprising if the parameters already get set in there, even if they will have to be updated for the next objective call.

So, I'd be fine with directly calling fill_in_parameters from within create_edatas.

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