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

Output file path not specified #9

Open
vpnu opened this issue Jun 9, 2023 · 3 comments
Open

Output file path not specified #9

vpnu opened this issue Jun 9, 2023 · 3 comments

Comments

@vpnu
Copy link

vpnu commented Jun 9, 2023

There is no way currently to get the output file path from the incoming crate. It is needed for the Wfexs stage file to be generated.

@beforan
Copy link

beforan commented Jun 12, 2023

We also need it to know what data entities to add to metadata / results on the execution CreateAction / hasPart on the root.

e.g. WfExS outputs a crate that contains quite a bit of stuff including but not limited to the outputs defined by the workflow and stage file.

When adding outputs to the main crate, I'm currently proposing we add that entire output in outputs/ but we don't necessarily want to reference all files in the metadata? (I'm not sure we are realistically capable of doing so).

Really, in addition to that folder in general as a Dataset, we should specifically add entities for outputs as defined by the Workflow, and so it would be easiest to get this from a CreateAction outputs definition. These would then be referenced in results and hasPart as per the spec.

@stain
Copy link
Member

stain commented Jun 12, 2023

I don't think all the outputs from WfExS should need to appear again in this crate if that is included as a sub-crate. It is fine to not reference all the files from a crate, you may just describe the folder for instance.

And then "lift" the particular outputs to the Trusted crate's Action - it's consumer do not need to know that those outputs are also in the nested "as-is" crate from WfExS. This is also probably easier as the nested action would have the different ID compared to the original request. (You can link those with sameAs in the top-level crate)

@vpnu
Copy link
Author

vpnu commented Jul 26, 2023

@stain We are currently trying to get the HutchAgent to write the Wfexs stage file using the incoming Trusted Workflow Run Crate plus the Workflow Run Crate from Workflow Hub once a DownloadAction has occured.
The current profile doesn't specify the input and output parameter values that are needed for the stage file to be generated, it only has their names. The values are in the secondary workflow, which is referenced by the main workflow but not explicitly mentioned in either the main incoming crate or the sub-crate.

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

3 participants