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 create_mission_definition and start_mission_exicution #39

Merged
merged 4 commits into from
Mar 10, 2023

Conversation

OleDrange
Copy link
Contributor

No description provided.

@OleDrange OleDrange force-pushed the missiondefinition branch 5 times, most recently from be5c9ef to 9159c58 Compare March 3, 2023 12:28
}

try:
response_dict: dict[str, Any] = self.client.query(mutation_string, params)
Copy link
Contributor

Choose a reason for hiding this comment

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

is it sufficient to say dict?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

dont know what you mean. It's the same as the other functions we have made. Il just let it stand and we can update it later maby

src/isar_exr/api/energy_robotics_api.py Outdated Show resolved Hide resolved
Copy link
Contributor

@GodVenn GodVenn left a comment

Choose a reason for hiding this comment

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

Some suggestions for naming and I think we will need to return the mission execution id

src/isar_exr/api/energy_robotics_api.py Outdated Show resolved Hide resolved
src/isar_exr/api/energy_robotics_api.py Outdated Show resolved Hide resolved
src/isar_exr/api/energy_robotics_api.py Outdated Show resolved Hide resolved
src/isar_exr/api/energy_robotics_api.py Outdated Show resolved Hide resolved
@GodVenn
Copy link
Contributor

GodVenn commented Mar 3, 2023

Can you add tests for these functions?
See the test for pause_current_mission for inspiration

@OleDrange OleDrange force-pushed the missiondefinition branch 4 times, most recently from 4b22f06 to 90c1cab Compare March 6, 2023 09:44
@OleDrange OleDrange requested a review from GodVenn March 6, 2023 09:47
@OleDrange OleDrange self-assigned this Mar 6, 2023
@OleDrange OleDrange added the feature New feature or request label Mar 6, 2023
Copy link
Contributor

@GodVenn GodVenn left a comment

Choose a reason for hiding this comment

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

Suggest renaming

tests/api/test_energy_robotics_api.py Outdated Show resolved Hide resolved
tests/api/test_energy_robotics_api.py Outdated Show resolved Hide resolved
@OleDrange OleDrange force-pushed the missiondefinition branch from 90c1cab to 18cef40 Compare March 6, 2023 12:50
@OleDrange OleDrange requested a review from GodVenn March 6, 2023 12:51
@OleDrange OleDrange force-pushed the missiondefinition branch from 18cef40 to 9e2cde6 Compare March 7, 2023 13:42
@OleDrange OleDrange force-pushed the missiondefinition branch from 9e2cde6 to 1cabb27 Compare March 9, 2023 12:23
Copy link
Contributor

@GodVenn GodVenn left a comment

Choose a reason for hiding this comment

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

LGTM

@OleDrange OleDrange merged commit 58ead1d into equinor:main Mar 10, 2023
@OleDrange OleDrange deleted the missiondefinition branch March 10, 2023 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants