-
Notifications
You must be signed in to change notification settings - Fork 28
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
Color variant support #2
Comments
The program should not run in just one phase
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: Ali Yousuf <notifications@github.com>
Sent: Wednesday, September 25, 2019 6:08:37 AM
To: alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Juan Camilo López Cortés. - Estudiante <juan.camilo.lopez@sgs.edu.co>; Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
^ @shonin<https://github.com/shonin> @Panda0003<https://github.com/Panda0003> @gary2817<https://github.com/gary2817>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=ALSTQUFM3YC6ZYJYBRR4HOLQLNBDLA5CNFSM4I2LHMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7RQI3A#issuecomment-534971500>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ALSTQUAPS32CAJJX2FK46EDQLNBDLANCNFSM4I2LHMAQ>.
|
I mean, that it should show first the first phase ñ, for example the red colors
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: Ali Yousuf <notifications@github.com>
Sent: Wednesday, September 25, 2019 6:08:37 AM
To: alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Juan Camilo López Cortés. - Estudiante <juan.camilo.lopez@sgs.edu.co>; Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
^ @shonin<https://github.com/shonin> @Panda0003<https://github.com/Panda0003> @gary2817<https://github.com/gary2817>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=ALSTQUFM3YC6ZYJYBRR4HOLQLNBDLA5CNFSM4I2LHMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7RQI3A#issuecomment-534971500>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ALSTQUAPS32CAJJX2FK46EDQLNBDLANCNFSM4I2LHMAQ>.
|
And you knit them
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: Juan Camilo López Cortés. - Estudiante
Sent: Wednesday, September 25, 2019 6:30:29 AM
To: alyyousuf7/Weaver <reply@reply.github.com>; alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
The program should not run in just one phase
Get Outlook for iOS<https://aka.ms/o0ukef>
________________________________
From: Ali Yousuf <notifications@github.com>
Sent: Wednesday, September 25, 2019 6:08:37 AM
To: alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Juan Camilo López Cortés. - Estudiante <juan.camilo.lopez@sgs.edu.co>; Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
^ @shonin<https://github.com/shonin> @Panda0003<https://github.com/Panda0003> @gary2817<https://github.com/gary2817>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=ALSTQUFM3YC6ZYJYBRR4HOLQLNBDLA5CNFSM4I2LHMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7RQI3A#issuecomment-534971500>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ALSTQUAPS32CAJJX2FK46EDQLNBDLANCNFSM4I2LHMAQ>.
|
Than with the blue and so on
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: Juan Camilo López Cortés. - Estudiante
Sent: Wednesday, September 25, 2019 6:31:05 AM
To: alyyousuf7/Weaver <reply@reply.github.com>; alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
And you knit them
Get Outlook for iOS<https://aka.ms/o0ukef>
________________________________
From: Juan Camilo López Cortés. - Estudiante
Sent: Wednesday, September 25, 2019 6:30:29 AM
To: alyyousuf7/Weaver <reply@reply.github.com>; alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
The program should not run in just one phase
Get Outlook for iOS<https://aka.ms/o0ukef>
________________________________
From: Ali Yousuf <notifications@github.com>
Sent: Wednesday, September 25, 2019 6:08:37 AM
To: alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Juan Camilo López Cortés. - Estudiante <juan.camilo.lopez@sgs.edu.co>; Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
^ @shonin<https://github.com/shonin> @Panda0003<https://github.com/Panda0003> @gary2817<https://github.com/gary2817>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=ALSTQUFM3YC6ZYJYBRR4HOLQLNBDLA5CNFSM4I2LHMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7RQI3A#issuecomment-534971500>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ALSTQUAPS32CAJJX2FK46EDQLNBDLANCNFSM4I2LHMAQ>.
|
In different phases
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: Juan Camilo López Cortés. - Estudiante
Sent: Wednesday, September 25, 2019 6:31:05 AM
To: alyyousuf7/Weaver <reply@reply.github.com>; alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
And you knit them
Get Outlook for iOS<https://aka.ms/o0ukef>
________________________________
From: Juan Camilo López Cortés. - Estudiante
Sent: Wednesday, September 25, 2019 6:30:29 AM
To: alyyousuf7/Weaver <reply@reply.github.com>; alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
The program should not run in just one phase
Get Outlook for iOS<https://aka.ms/o0ukef>
________________________________
From: Ali Yousuf <notifications@github.com>
Sent: Wednesday, September 25, 2019 6:08:37 AM
To: alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Juan Camilo López Cortés. - Estudiante <juan.camilo.lopez@sgs.edu.co>; Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
^ @shonin<https://github.com/shonin> @Panda0003<https://github.com/Panda0003> @gary2817<https://github.com/gary2817>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=ALSTQUFM3YC6ZYJYBRR4HOLQLNBDLA5CNFSM4I2LHMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7RQI3A#issuecomment-534971500>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ALSTQUAPS32CAJJX2FK46EDQLNBDLANCNFSM4I2LHMAQ>.
|
I also think so, multiple colors follow the smooth, but how does this color distinguish? |
We could use image recognition
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: gary2817 <notifications@github.com>
Sent: Wednesday, September 25, 2019 7:02:23 AM
To: alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Juan Camilo López Cortés. - Estudiante <juan.camilo.lopez@sgs.edu.co>; Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
I also think so, multiple colors follow the smooth, but how does this color distinguish?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=ALSTQUFYV3SJSIBLIG6QYQDQLNHM7A5CNFSM4I2LHMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7RUNXY#issuecomment-534988511>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ALSTQUCO3BT4KZ76OSPEMH3QLNHM7ANCNFSM4I2LHMAQ>.
|
Using tensorflow
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: gary2817 <notifications@github.com>
Sent: Wednesday, September 25, 2019 7:02:23 AM
To: alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Juan Camilo López Cortés. - Estudiante <juan.camilo.lopez@sgs.edu.co>; Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
I also think so, multiple colors follow the smooth, but how does this color distinguish?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=ALSTQUFYV3SJSIBLIG6QYQDQLNHM7A5CNFSM4I2LHMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7RUNXY#issuecomment-534988511>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ALSTQUCO3BT4KZ76OSPEMH3QLNHM7ANCNFSM4I2LHMAQ>.
|
Oh wow @Panda0003, I’m getting many notifications. Can you please compile a complete message and then send? And yes, we need to separate out colors from an image, let’s say 4 colors: Black, Blue, Yellow, Red. (These colors are used in the link mentioned in the top comment) Then we use the same strategy to detect weave pattern for individual colors and overlay individual colors on top of each other. |
Wow, it’s great. |
So you didnt change the code, except for the color of the line. Also you changed the image processed to different parts.
…________________________________
De: Ali Yousuf <notifications@github.com>
Enviado: jueves, 26 de septiembre de 2019 9:57 a. m.
Para: alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Juan Camilo López Cortés. - Estudiante <juan.camilo.lopez@sgs.edu.co>; Mention <mention@noreply.github.com>
Asunto: Re: [alyyousuf7/Weaver] Color variant support (#2)
This is what I have got so far:
[Screenshot 2019-09-26 at 7 54 27 pm]<https://user-images.githubusercontent.com/14050128/65699402-79d20000-e097-11e9-91b8-148e97816ee7.png>
The result on the left can improve ALOT! This is just my first attempt at setting the correct colors, I got so excited to share the work-in-progress!
Right now, I'm manually separating out colors from the original image using Photoshop and saving it into 4 new images, and then running the same algorithm on all 4 images one by one.
It contains 2000 lines for each color, making it 8000 lines.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=ALSTQUF3VQ55SZPKJZHZMS3QLTEV5A5CNFSM4I2LHMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7V4AGI#issuecomment-535543833>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ALSTQUA7VPKMS53WULTRLJDQLTEV5ANCNFSM4I2LHMAQ>.
|
Its great but i think we use more colors
…________________________________
De: gary2817 <notifications@github.com>
Enviado: jueves, 26 de septiembre de 2019 10:07 a. m.
Para: alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Juan Camilo López Cortés. - Estudiante <juan.camilo.lopez@sgs.edu.co>; Mention <mention@noreply.github.com>
Asunto: Re: [alyyousuf7/Weaver] Color variant support (#2)
Wow, it’s great.
The first time can be so good
If you can be clearer, it will be even better.
I think the image you are looking for contains as few colors as possible.
This is better to separate.
And can make the image more vivid
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=ALSTQUGRFGIPUKPES3FLELLQLTFZ5A5CNFSM4I2LHMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7V5D2I#issuecomment-535548393>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ALSTQUBT2K4NXVBUUP6KIC3QLTFZ5ANCNFSM4I2LHMAQ>.
|
I have completely rewritten the code to allow many more improvements. Previously, the code was pretty procedural, now i have broken things down to follow OOP. I'll merge the code once I clean up the rest of the code. |
You should give a try to image recognition
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: Ali Yousuf <notifications@github.com>
Sent: Thursday, September 26, 2019 3:46:43 PM
To: alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Juan Camilo López Cortés. - Estudiante <juan.camilo.lopez@sgs.edu.co>; Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
I have completely rewritten the code to allow many more improvements. Previously, the code was pretty procedural, now i have broken things down to follow OOP.
I'll merge the code once I clean up the rest of the code.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=ALSTQUHTQ2J5KGFVWX547CLQLUNTHA5CNFSM4I2LHMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7W5EUI#issuecomment-535679569>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ALSTQUH7HF5V7ELSRFK576TQLUNTHANCNFSM4I2LHMAQ>.
|
Btw, could you explain to me all the code: the old one, please. Thank you. Ill try to work using color channels.
…________________________________
De: Ali Yousuf <notifications@github.com>
Enviado: jueves, 26 de septiembre de 2019 3:46 p. m.
Para: alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Juan Camilo López Cortés. - Estudiante <juan.camilo.lopez@sgs.edu.co>; Mention <mention@noreply.github.com>
Asunto: Re: [alyyousuf7/Weaver] Color variant support (#2)
I have completely rewritten the code to allow many more improvements. Previously, the code was pretty procedural, now i have broken things down to follow OOP.
I'll merge the code once I clean up the rest of the code.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=ALSTQUHTQ2J5KGFVWX547CLQLUNTHA5CNFSM4I2LHMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7W5EUI#issuecomment-535679569>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ALSTQUH7HF5V7ELSRFK576TQLUNTHANCNFSM4I2LHMAQ>.
|
@Panda0003, the explanation is in the |
Signed-off-by: Ali Yousuf <aly.yousuf7@gmail.com>
Wow |
Just run the application as usual. I have updated the README with new instructions. And no, you don’t need to process images before. But there is capability to process it manually if you want to and give it to the software. I understand that this tool right now requires basic programming skills to play with it. |
Greatttt
Get Outlook for iOS<https://aka.ms/o0ukef>
…________________________________
From: Ali Yousuf <notifications@github.com>
Sent: Monday, September 30, 2019 11:09:15 AM
To: alyyousuf7/Weaver <Weaver@noreply.github.com>
Cc: Juan Camilo López Cortés. - Estudiante <juan.camilo.lopez@sgs.edu.co>; Mention <mention@noreply.github.com>
Subject: Re: [alyyousuf7/Weaver] Color variant support (#2)
Just run the application as usual.
I have updated the README with new instructions.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub<#2?email_source=notifications&email_token=ALSTQUFNMXYM4CM2LEQEWD3QMIQCXA5CNFSM4I2LHMA2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD76GECY#issuecomment-536633867>, or mute the thread<https://github.com/notifications/unsubscribe-auth/ALSTQUAOCJTSZSLORYWBQJTQMIQCXANCNFSM4I2LHMAQ>.
|
This is excellent! Apparently I don't look at my github notifications enough. I'll do a full look through the code when I'm able. |
Hi |
@alyyousuf7 Hi ! I was wondering, is there any way to pick up the color sequence from the last knit ? What I mean is, for example: I start by weaving the black lines, then the white ones, and then the yellow ones, then I want to continue with the black ones, continuing from the last knitted point. |
Yes, you can change the order of color from example.pde where they are defined. |
@alyyousuf7 Yes, i already did it, but the code starts again by overlapping the black lines with the previous ones, I wonder if it is possible instead to continue with the sequence of black lines from the last previous point knitted. You know what I mean? |
@Enomis9339, it always starts the new color from point zero. It shouldn't matter much because usually the number of lines of each color is so high. |
@alyyousuf7 What I mean is that, in my opinion, it has reached a satisfactory result if the layers of a single color could alternate with the others continuously. |
Oh, you mean that we alternate the colors after every few lines to get a better result. Is that what you mean? |
@alyyousuf7 Exactly! the ideal would be to superimpose one line at a time the different colors, but this in reality would take an absurd time, so why not try to change color only after a series of lines to be defined? Reduce the time of realization and also the error. |
It's a great idea and might get us better results! I'll try to work on it (i'm quite busy these days though). I'm not sure if that's possible with the current implementation, it will need a bit tweaking to support a layer continuing from middle. |
@Enomis9339, I was thinking about the solution, and I think there will be some minor issues. For example, when you define each color with the number of lines, those numbers represents the maximum number of lines for that color, that means it’s not necessary that each color has same number of lines, each color may have less number of lines than other color. So at one point, superimposing will stop working. Nevertheless, it looks like that it will give better result. And I think it can be done with minor changes. We just need to change the logic to draw lines in different order. |
@alyyousuf7 You got it right:) It is logical that there will be more lines of one color than another, It all depends on how much color is present in the image of course. So if we could create in the program the change I proposed, then it might be possible, by trying, to find the right number of lines of a color that leads to the best result. |
@alyyousuf7 I was still thinking about the logic we could use to get the desired result. I was wondering if it is better to:
Insert multiple layers as above, but make sure that next layer of the same color starts from the last point to which the line of the previous layer of the same color was connected. I don't know which one is more logical and easy to do. |
If you look at the implementation of MultipleImageLayer class, it takes n images and n configs. If we modify it to accept pointer of images (instead of images), we can make it work to use the same old layer images for newer layer configs. Suppose we have 3 colors to weave with, red, green, and blue. But we want to repeat the layer again, so we might have 6 or 9 or 12 configs (multiple of the number of colors). We can create a function similar to “createColorLayer”, which will instead of mapping one config with one image, we will map multiple configs for a single image, and repeat this for each color. (Look at the last line of the createColorLayer function) What this will do is, suppose if in the first configs, the color is red, it will draw 50 lines for it, and then move on to the next color. But on the 4th config, the color will be red again, but instead of the initial image for red, it will have the images on which the first config had already worked on* (due to pointer to the same PImage). *This requires you to understand that the original algorithm actually works by “subtracting” the color line by line. I’ve tried to explain my best, but I’m sure you might not understand completely. Feel free to question. |
I admit I have not fully understood, unfortunately my processing knowledge is limited to the basic tutorials that I followed on the internet. However, I continue to be convinced that this logic could lead to a better result. |
@alyyousuf7 100 lines back 100 lines each layer just so good! Later I'll upload my handle image |
I can't wait to see the result :D |
Have you edited the previous code posted by @alyyousuf7 ? |
@kitor7, I'd also like to see the screenshot of what the simulation showed you. That's how I judge that what it might look like in real life. |
@khuongtrung936, please choose the input image with higher resolution. |
Can you create a new ticket for that please? |
could you optimize rgb version |
Could you code your idea? |
hi. how to change image in code? in folder are black,red,blue,yellow. is this is a color chanels? or what is it. how to replace this images? |
Many people have requested color variant support on Weaver.
https://www.saatchiart.com/art/New-Media-Color-knit-5-Limited-Edition-of-4/858823/6200223/view
Let's discuss here the possible solution to the issue.
Some of the discussion are in #1, and from now on we'll be using this issue to follow up.
[Update]
You can create color variant output using Example03 code in the master branch. You have to choose the colors very meticulously. It can be a real pain to choose correct colors.
Keeping the issue opened until we find a better approach to automatically pick colors. [I hope that's possible]
The text was updated successfully, but these errors were encountered: