search

ohsu-comp-bio / py-tes

Python SDK for the GA4GH Task Execution API
https://py-tes.readthedocs.io/
MIT License
5 stars 8 forks source link

Handle additional properties #64

Open athith-g opened 1 month ago

athith-g commented 1 month ago

Currently, py-tes throws errors when encountering properties that are not in the TES specs. For example, when using py-tes v0.4.2 with funnel v0.11.0, the presence of ignore_error in the response body produces the following traceback (from proTES #176):

Screenshot 2024-06-07 at 3 34 46 PM

However, since additionalProperties is not false in the TES specs, any additional spec provided should be handled without throwing an error.

uniqueg commented 1 month ago

Thanks for reporting @athith-g. I agree that py-tes should handle additional props gracefully - the current implementation is not spec-compliant.