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

[uss_qualifier] Add flight intent transformations #400

Merged
merged 3 commits into from
Dec 15, 2023

Conversation

BenjaminPelletier
Copy link
Member

Currently, our flight intents are fairly cumbersome to manipulate and therefore it is difficult to define test configurations in new areas. This PR introduces flight intent transformations which allows basic/standard flight intents to be easily moved into appropriate areas. Flight intents specific to CHE and US and removed in favor of "standard" flight intents which are then offset into CHE and US in the appropriate resources. The f3548_self_contained configuration is oriented specifically toward the US while the uspace configuration remains oriented toward CHE.

Comment on lines 94 to 96
from loguru import logger

logger.warning("Applying transformations")
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
from loguru import logger
logger.warning("Applying transformations")

nit: I guess this could be removed

Copy link
Member Author

Choose a reason for hiding this comment

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

Oops, definitely. Thanks.

@BenjaminPelletier BenjaminPelletier merged commit 28ae6a2 into interuss:main Dec 15, 2023
9 checks passed
@BenjaminPelletier BenjaminPelletier deleted the translate-to branch December 15, 2023 02:08
github-actions bot added a commit that referenced this pull request Dec 15, 2023
* Add flight intent transformations

* Remove debugging message 28ae6a2
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.

2 participants