SED-ML / KiSAO

Ontology of algorithms for analyzing biological models, their parameters, and their outputs
Artistic License 2.0
9 stars 0 forks source link

SED-ML Aggregation function (New terms for count, stddev, variance) #93

Closed jonrkarr closed 2 years ago

jonrkarr commented 3 years ago
matthiaskoenig commented 3 years ago

would be great in addition

matthiaskoenig commented 3 years ago

Can we add the remaining functions under https://bioportal.bioontology.org/ontologies/KISAO/?p=classes&conceptid=http%3A%2F%2Fwww.biomodels.net%2Fkisao%2FKISAO%23KISAO_0000824? Is there a documentation how to update terms? I could add these myself to get up to speed how to do this.

It would be amazing if we could change the labels so that we have:

which is in line with how this normally works. I.e. the ignoring NaN functions are special versions of the normal functions.

jonrkarr commented 3 years ago

I put a brief guide to editing the ontology here: https://github.com/SED-ML/KiSAO/blob/dev/CONTRIBUTING.md#modifying-the-ontology. Protege is a good editing tool. This makes it easy to see the relationships as you add terms. The existing terms are also a good reference.

Please also add a note about your changes to CHANGELOG.md.

The GitHub action will automatically run the few tests.

I can take of releasing your changes as v2.22.

jonrkarr commented 3 years ago

I agree with labels such as mean and mean (ignoring NaN). This is similar to the names of NumPy functions (mean, nanmean) and the default configuration of MATLAB (mean(..., nanflag)). The "default" should be to include NaN such the result of a mean over a NaN is NaN.

matthiaskoenig commented 2 years ago

The following terms have been added/updated. Pull request shortly: