Skip to content

Conversation

@ChristofferEdlund
Copy link
Contributor

Problem

No pytests for the new albumentations transform in darwin-py transforms

Solution

Added a pytest

Changelog

  • Pytest added for albumentations transform

@linear
Copy link

linear bot commented Sep 29, 2023

AI-1245 Add transformation albumentations test in darwin-py

Add transformation albumentations test in darwin-py

ChristofferEdlund

This comment was marked as outdated.

Copy link
Contributor

@owencjones owencjones left a comment

Choose a reason for hiding this comment

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

One change requested, also reach out on slack to sort the issue with albumentations

Copy link
Contributor

@owencjones owencjones left a comment

Choose a reason for hiding this comment

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

Looks good, good test coverage with happy and sad paths

Copy link
Contributor

@owencjones owencjones left a comment

Choose a reason for hiding this comment

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

Yeah, all looking good. Ready for QA.

"python.formatting.blackPath": "black",
"python.formatting.provider": "none",
"python.formatting.provider": "black",
"python.formatting.blackArgs": [
Copy link
Contributor

Choose a reason for hiding this comment

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

Removing these lines below on here will mean that formatOnSave starts working better

@ChristofferEdlund ChristofferEdlund merged commit 56968b3 into master Oct 16, 2023
@ChristofferEdlund ChristofferEdlund deleted the ai-1245-add-transformation-albumentations-test-in-darwin-py branch October 16, 2023 11:35
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.

3 participants