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

add alternative executionUnit formats to support OGC-API schemas #574

Merged
merged 6 commits into from
Oct 13, 2023

Conversation

fmigneault
Copy link
Collaborator

changes

  • Support alternative representations from OGC API - Processes schemas for executionUnit definition
    during Process deployment. The unit does not need to be nested under unit or a list anymore, and can instead
    be directly provided as JSON mapping. For backward compatibility, the previous list representation is still allowed
    (fixes Support new formats of executionUnit #507).
  • Support an additional type property along a unit item describing an executionUnit to specify an IANA
    Media-Type that categories the unit contents, similarly to how it could be provided for its href counterpart.
    For the moment, only CWL-based unit are supported, but this could allow future extensions to provide alternate
    representations of an Application Package.

@fmigneault fmigneault self-assigned this Oct 13, 2023
@github-actions github-actions bot added ci/doc Issue related to documentation of the package ci/tests Tests of the package and features feature/oas Issues related to OpenAPI specifications. process/wps3 Issue related to WPS 3.x (REST-JSON) processes support labels Oct 13, 2023
@codecov
Copy link

codecov bot commented Oct 13, 2023

Codecov Report

Merging #574 (1767a00) into master (7413e61) will decrease coverage by 0.02%.
Report is 1 commits behind head on master.
The diff coverage is 94.11%.

❗ Current head 1767a00 differs from pull request most recent head e9957f7. Consider uploading reports for the commit e9957f7 to get more accurate results

@@            Coverage Diff             @@
##           master     #574      +/-   ##
==========================================
- Coverage   84.90%   84.88%   -0.02%     
==========================================
  Files          79       79              
  Lines       17935    17921      -14     
  Branches     2718     2718              
==========================================
- Hits        15227    15213      -14     
  Misses       1980     1980              
  Partials      728      728              
Files Coverage Δ
weaver/wps_restapi/swagger_definitions.py 99.83% <100.00%> (-0.01%) ⬇️
weaver/processes/utils.py 80.00% <83.33%> (+0.14%) ⬆️

... and 2 files with indirect coverage changes

@fmigneault fmigneault merged commit 10b1403 into master Oct 13, 2023
26 of 28 checks passed
@fmigneault fmigneault deleted the alt-exec-unit branch October 13, 2023 23:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/doc Issue related to documentation of the package ci/tests Tests of the package and features feature/oas Issues related to OpenAPI specifications. process/wps3 Issue related to WPS 3.x (REST-JSON) processes support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Support new formats of executionUnit
1 participant