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.

Sign up for GitHub

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

Jump to bottom

Task-level remote input localization #2

Open
sitapriyamoorthi opened this issue Dec 16, 2024 · 3 comments
Open

Task-level remote input localization #2

sitapriyamoorthi opened this issue Dec 16, 2024 · 3 comments
Assignees
@sitapriyamoorthi
Labels
unit test Adding or modifying a unit test

Comments

@sitapriyamoorthi
Copy link
Collaborator

sitapriyamoorthi commented Dec 16, 2024 •
edited
Loading

  1. If inputs at the task level are being correctly localized from remote (non-execution folder) and made available to a task and are successfully copied into the execution folder
  2. Also testing the use of string, integer, float and boolean as inputs to a task
  3. Also test if the output from one task is properly being made available for use as an input in another task
  4. If task outputs are localized to the execution directory
@sitapriyamoorthi sitapriyamoorthi self-assigned this Dec 16, 2024
@seankross seankross added the unit test Adding or modifying a unit test label Dec 18, 2024
@sitapriyamoorthi sitapriyamoorthi changed the title Input Handling (Validation and Required Inputs) Task-level remote input localization Dec 20, 2024
@sitapriyamoorthi
Copy link
Collaborator Author

While this unit test addresses localization of file inputs we need to address what handles the scenario for files stored in fast or temp:

@sitapriyamoorthi
Copy link
Collaborator Author

In the future we should write a test to evaluate what happens if you dont have access.

@sitapriyamoorthi
Copy link
Collaborator Author

DO NOT MERGE ASSOCIATED BRANCH TO MAIN... NEEDS PROOF ACCESS

@sitapriyamoorthi sitapriyamoorthi linked a pull request Jan 17, 2025 that will close this issue
2 task level remote input localization #69
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sitapriyamoorthi sitapriyamoorthi

Labels
unit test Adding or modifying a unit test
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

2 task level remote input localization FredHutch/wdl-unit-tests
2 participants
@seankross @sitapriyamoorthi