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

PEtab SciML TODO #11

Open
3 of 8 tasks
sebapersson opened this issue Dec 18, 2024 · 4 comments
Open
3 of 8 tasks

PEtab SciML TODO #11

sebapersson opened this issue Dec 18, 2024 · 4 comments

Comments

@sebapersson
Copy link
Owner

sebapersson commented Dec 18, 2024

This issue acts as a TODO on things to complete for the standard:

  • Add different initializationPriors. In addition the the supported priors in PEtab we should add glorot_normal, glorot_uniform as well as kaiming_normal and kaiming_uniform, where for the latter users will have to specify the gain. Need to add test cases for this.
  • Add ability to set initialization for a layer via the parameters table. The following should be allowed in the parameters table: netId.layerId. Need to add test cases for this.
  • Add test case where neural network parameters are constant (not estimated).
  • Following PEtab YAML formatting #4, specify neural network output in the condition table. This will wait for PEtab v2 spec completion.
  • Update the specification for PEtab SciML, and host it online in this repository. Usually I use Julia Documenter.jl for hosting docs, but I guess we should use something Python based for consistency?
  • Update to use the code in src/python/petab_sciml for setting up the test cases.
  • Add repository tests, specifically add tests to test the consistency between Lux.jl and PyTorch (this is already done in the net test-cases, but this should be refactored to a proper test directory).
  • Add and test for the yaml model specification and parameter import with PyTorch?

Feel free to add any more points you might find relevant. Once the above points are addressed, I think the extension should be close to complete.

@FFroehlich
Copy link
Contributor

FFroehlich commented Dec 19, 2024

sorry to be that guy, but in the testcases

  • model_files in the yaml files needs to be a key value pairs with the model id as key
  • petabEntityId and modelEntityId in the mapping tables are swapped

@sebapersson
Copy link
Owner Author

sebapersson commented Dec 19, 2024

sorry to be that guy, but in the testcases

Thanks for noticing, at least this is easy to fix. I will look at this tomorrow. As for the model id, in this case should it match the id in the SBML file? (SBML model name)

@FFroehlich
Copy link
Contributor

FFroehlich commented Dec 19, 2024

sorry to be that guy, but in the testcases

Thanks for noticing, at least this is easy to fix. I will look at this tomorrow. As for the model id, in this case should it match the id in the SBML file? (SBML model name)

I don't think the model id is used anywhere at this point (but could be referred to in modelEntityId), so the value is not important

@sebapersson sebapersson mentioned this issue Dec 20, 2024
@sebapersson
Copy link
Owner Author

@FFroehlich the formatting issues are now fixed with #12

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

No branches or pull requests

2 participants