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_time_derived_measurements breaks if you use _script in the meds_transform_runner #202

Open
Oufattole opened this issue Sep 3, 2024 · 1 comment
Labels
bug Something isn't working priority:medium A medium priority issue. Runner For things about the multi-stage, single-script Runner

Comments

@Oufattole
Copy link
Collaborator

for feature_name, feature_cfg in stage_cfg.items():

This will fail as _script will be a feature_name in stage_cfg.items()

@mmcdermott

@mmcdermott
Copy link
Owner

Thanks for catching this! You actually don't need to include _script in the runner for things where the stage name will be the same as the script name, I think -- it should automatically determine the script for a stage called add_time_derived_measurements.py, but either way the _script param shouldn't propagate into the actual stage_cfg.

@mmcdermott mmcdermott added bug Something isn't working priority:medium A medium priority issue. Runner For things about the multi-stage, single-script Runner labels Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority:medium A medium priority issue. Runner For things about the multi-stage, single-script Runner
Projects
None yet
Development

No branches or pull requests

2 participants