smileyface12349 / itinerant-iterators

Python Discord Code Jam 2023
MIT License
0 stars 0 forks source link

Distribute bit encoding uniformly during steganography #9

Open jj-style opened 1 year ago

jj-style commented 1 year ago

With the current implementation of least significant bit encoding it encodes the data in each colour channel in each pixel starting top left. Sharp eyed people may notice a bit of corruption especially if that part of the image should be all the same colour.

We should either:

I like the second option more as it's a bit more general and the pixel generators could be reused in other steganography implementations

smileyface12349 commented 1 year ago

I'll take this one

jj-style commented 1 year ago

Nice one yeah go for it. Just changing the current implementation to use PIL not opencv so will try push that up this evening

smileyface12349 commented 1 year ago

Ah, is it going to be conflict hell if I start now?

jj-style commented 1 year ago

if you start a new file alongside LSB and don't touch steganography.py or lsb.py you should be okay. Minor change to the Image class but otherwise should be okay. Will try get it pushed up ASAP after testing so doesn't cause too many conflicts if you make a start now

smileyface12349 commented 1 year ago

Ok I'll wait until you're done. No rush.

jj-style commented 1 year ago

Should be good to go now!