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 logo #11

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

Add logo #11

wants to merge 1 commit into from

Conversation

benjamingr
Copy link

No description provided.

@@ -1,4 +1,6 @@
lesson02
========

![](https://www.loacker.com/uploads/pics/header_bildgalerie_welt_der_waffeln_classic_napolitaner_03.jpg?is_ilan=true&devour=yes)
Copy link
Owner

Choose a reason for hiding this comment

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

Where is the copyright def for this image?

Copy link
Author

Choose a reason for hiding this comment

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

Inside, it is embedded - please check

Copy link
Owner

Choose a reason for hiding this comment

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

Can you please add an option to modify the image via the UI?
E.g. rotate, run a fourier transform etc, would be really helpful!

Copy link
Author

Choose a reason for hiding this comment

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

Sure, please provide instructions on how to do so - I will gladly contribute further.

Copy link
Owner

Choose a reason for hiding this comment

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

Create a ui with an input
Then send the user request via sockets over UDP to the server
In the server create a worker which will parse the request and run the logic and will return the answer.
I'll be happy to have a technical design review with you once you will have a proper design.

Copy link

Choose a reason for hiding this comment

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

Its just a logo!

Copy link
Owner

Choose a reason for hiding this comment

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

I want it to be feature complete

Copy link
Author

Choose a reason for hiding this comment

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

@ilantc can you please make the skeleton for me to fill the stuff in? I am looking forward to collaborating on this!

Copy link
Owner

Choose a reason for hiding this comment

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

@benjamingr feel free to create one :)
#13

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