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 automatic image detection #264

Merged
merged 6 commits into from
Mar 9, 2023
Merged

Conversation

jannikbecher
Copy link
Contributor

This is my first time contributing to this project, and I'm excited to make my first pull request. Please let me know if there's anything I can improve!

Closes #43.

Detects image type by its magic number and renders the image as a tabbed output.
kino_pr_add_automatic_image_detection

Detects the mime type of a raw image and renders it.
Currently implemented for jpeg, png, svg and gif
lib/kino/utils.ex Outdated Show resolved Hide resolved
lib/kino/utils.ex Outdated Show resolved Hide resolved
@jonatanklosko
Copy link
Member

@jannikbecher fantastic, thanks for the PR!

@jannikbecher
Copy link
Contributor Author

@jonatanklosko thanks for this great framework! It was a pure joy try to figure out what needs to be changed :)

@jannikbecher
Copy link
Contributor Author

sorry for the commit mess..
still learning the pull request workflow

@jonatanklosko
Copy link
Member

No worries, take your time. We will squash it all into a single commit when merged anyway, so it's fine :)

@jannikbecher
Copy link
Contributor Author

Should be good now, thanks for the patience :)
One question: do you use the web interface for code reviews?

Copy link
Member

@jonatanklosko jonatanklosko left a comment

Choose a reason for hiding this comment

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

Perfect!

One question: do you use the web interface for code reviews?

Yes :)

@jonatanklosko jonatanklosko merged commit 5222ebd into livebook-dev:main Mar 9, 2023
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.

Automatic image detection
2 participants