Skip to content
This repository has been archived by the owner on Aug 27, 2024. It is now read-only.

test: add tests for image preparation conversion #143

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stijnmoreels
Copy link
Member

Add tests for the image preparation conversion.

@netlify
Copy link

netlify bot commented Sep 26, 2022

Deploy Preview for arcus-ml canceled.

Name Link
🔨 Latest commit 4172fd4
🔍 Latest deploy log https://app.netlify.com/sites/arcus-ml/deploys/63316959b1b33c0009883b0c

@stijnmoreels
Copy link
Member Author

Still missing a single code execution path for the property tests, and that's the path where the image preparation uses the np.dot function, but I'm having trouble understanding both what it does and why the current implementation uses a fixed value: [0.3, 0.59, 0.11].

@stijnmoreels
Copy link
Member Author

And apparently, the tests now fail bc the matplotlib doesn't have an Axes class anymore with numColls properties. 🤔 Wondering if we use a fixed version during testing.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants