Skip to content

[uss_qualifier] Add ParallelFlightPlannerCombinations, parallel execution of actions generator and use everyting in f3548_21#1470

Open
the-glu wants to merge 4 commits into
interuss:mainfrom
Orbitalize:1458_ParallelActionGenerator
Open

[uss_qualifier] Add ParallelFlightPlannerCombinations, parallel execution of actions generator and use everyting in f3548_21#1470
the-glu wants to merge 4 commits into
interuss:mainfrom
Orbitalize:1458_ParallelActionGenerator

Conversation

@the-glu
Copy link
Copy Markdown
Contributor

@the-glu the-glu commented May 20, 2026

This PR is stacked on top of #1468 and contrib to #1458

'Core' implementation of parallelization, focusing on f3548_21

A few remarks:

  • I used thread for simplicity instead of processed.
  • This make some test steps wrong. Example: PrepareFlightPlanners is supposed to clear area, but it won't use modified flight intents, only the base one. Not sure how to fix that.
  • Some tests have not been changed to parallel mode, as they don't / won't be able to handle it. Eg. GetOpResponseDataValidationByUSS is checking that all notification are expected, but that not the case dues when others tests are running.
  • No changelog entries, but this version is probably worth a warning dues to new resources need.
    • I tried to make the FlightIntentsModifier ones as local resources, but local resources cannot reference 'global' ones. This could also be fixed by improving resource initialization and moving *_parallel defintions in f3548_21.yaml

@the-glu the-glu force-pushed the 1458_ParallelActionGenerator branch 4 times, most recently from 7add034 to 29b2ac4 Compare May 20, 2026 11:46
context.parallel_marker = f" [{i + 1}/{len(branches)}]"
return run_branch(branch)

with ThreadPoolExecutor(max_workers=min(len(branches), 8)) as ex:
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NB: Not a setting, probably worth a settting in future PRs.

@the-glu the-glu force-pushed the 1458_ParallelActionGenerator branch from 29b2ac4 to 6aa0e47 Compare May 20, 2026 12:51
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

Successfully merging this pull request may close these issues.

1 participant