search

mqchau / EECS22L_Project2

GNU General Public License v3.0
0 stars 0 forks source link

Alpha Release #3

Closed mqchau closed 10 years ago

mqchau commented 10 years ago

Hi all, Please find below the description of Alpha Release. Our goal here is to earn all basic credit for Alpha Release + small extra credit of supporting extra images format. You'll need to read so you don't develop in wrong direction. I know it's long but there's a lot of empty spaces.

https://drive.google.com/a/uci.edu/file/d/0B6hFDKbQXelUamtkdnFMZllhaHM/edit?usp=sharing

Quan.

antrinh commented 10 years ago

Oh wow, thanks for the clarification Quan T_T

On Sat, Mar 1, 2014 at 7:27 PM, mqchau notifications@github.com wrote:

Andrew, you should just focus on determine only 1 image from Hanchel. Don't worry about order for now. Order will just be an upper level function iterating through every image, calling a lower level function comparing each images with the fonts database.

On Sat, Mar 1, 2014 at 4:14 PM, hanchelc notifications@github.com wrote:

I just realized when I commented out my debugging printfs, I might've missed the ' if ' statements before them. that'll fk up the code if you try to run it.

anyway, uhhh... how do you know what order? test0, test1, ..., testn will go in the order Left to Right, Up to Down, starting from A. Do you get what I mean?

Reply to this email directly or view it on GitHub< https://github.com/mqchau/EECS22L_Project2/issues/3#issuecomment-36441436>

.

Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/3#issuecomment-36444888 .

jamiesl commented 10 years ago

I've been working on a branch created on my machine and tried to push it to GitHub, but nothing showed up. Do I have to merge with master for these changes to show or something?

mqchau commented 10 years ago

No, you do this: git push origin On git hub you will select your branch to see it

On Sat, Mar 1, 2014 at 10:10 PM, jamiesl notifications@github.com wrote:

I've been working on a branch created on my machine and tried to push it to GitHub, but nothing showed up. Do I have to merge with master for these changes to show or something?

Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/3#issuecomment-36447238 .

jamiesl commented 10 years ago

Wow, took me 10 minutes to figure out how to push everything over. I'm not sure how to test with everything all together, so should I just merge with master? I tested my function separately before putting it into Model.c, and it seemed to work fine

mqchau commented 10 years ago

Leave merge to me, just show me here what files you need to compile and link

On Sat, Mar 1, 2014 at 10:26 PM, jamiesl notifications@github.com wrote:

Wow, took me 10 minutes to figure out how to push everything over. I'm not sure how to test with everything all together, so should I just merge with master? I tested my function separately before putting it into Model.c, and it seemed to work fine

Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/3#issuecomment-36447487 .

jamiesl commented 10 years ago

Okay, on my branch (ReadImage) I added Image.h and .c from the 22 solutions last quarter and added my function (IMAGE * ReadImage) to Model.h and Model.c. Everything from my machine should be pushed onto that branch already

mqchau commented 10 years ago

Ok, thanks

On Sat, Mar 1, 2014 at 10:31 PM, jamiesl notifications@github.com wrote:

Okay, on my branch (ReadImage) I added Image.h and .c from the 22 solutions last quarter and added my function (IMAGE * ReadImage) to Model.h and Model.c. Everything from my machine should be pushed onto that branch already

Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/3#issuecomment-36447552 .

mqchau commented 10 years ago

Jamie, Your functions works ok with jpeg + png + gif + ppm. However, it DOESN'T work with bmp or pgm, maybe also pbm. For Alpha release I think it's OK for now. But for Beta release I want all those 3 formats to work as well. Quan.

On Sat, Mar 1, 2014 at 10:35 PM, Quan Quoc Minh Chau mqchau@uci.edu wrote:

Ok, thanks

On Sat, Mar 1, 2014 at 10:31 PM, jamiesl notifications@github.com wrote:

Okay, on my branch (ReadImage) I added Image.h and .c from the 22 solutions last quarter and added my function (IMAGE * ReadImage) to Model.h and Model.c. Everything from my machine should be pushed onto that branch already

Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/3#issuecomment-36447552 .

mqchau commented 10 years ago

Eric's Rotate + Crops woks fine. There is only one thing in rotate not OK: the outside is dark. Eric please make the outside of the picture to be white since we'll be dealing with paper the outside must be white. Otherwise hanchel's function won't work.

So far I haven't enable OCR function yet. Only open image + crop + rotate + black white. Will try OCR tomorrow with merge from Hanchel + Andrew.

Hanchel, could you pleae tell me which functions do I call from you and which files do I need to compile to run your functions? Andrew, please finish it quick.

Quan.

On Sat, Mar 1, 2014 at 11:02 PM, Quan Quoc Minh Chau mqchau@uci.edu wrote:

Jamie, Your functions works ok with jpeg + png + gif + ppm. However, it DOESN'T work with bmp or pgm, maybe also pbm. For Alpha release I think it's OK for now. But for Beta release I want all those 3 formats to work as well. Quan.

On Sat, Mar 1, 2014 at 10:35 PM, Quan Quoc Minh Chau mqchau@uci.eduwrote:

Ok, thanks

On Sat, Mar 1, 2014 at 10:31 PM, jamiesl notifications@github.comwrote:

Okay, on my branch (ReadImage) I added Image.h and .c from the 22 solutions last quarter and added my function (IMAGE * ReadImage) to Model.h and Model.c. Everything from my machine should be pushed onto that branch already

Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/3#issuecomment-36447552 .

mqchau commented 10 years ago

Alrite, atleast now we have some one cvs commit on school server. I could clone from it into my personal computer (ubuntu virtual machine) and commit from my virtual machine. Andrew, you promised to finished on yesterday night, where's your submission? Hanchel, could you give me short description how to compile your code? Quan.

On Sun, Mar 2, 2014 at 12:02 AM, Quan Quoc Minh Chau mqchau@uci.edu wrote:

Eric's Rotate + Crops woks fine. There is only one thing in rotate not OK: the outside is dark. Eric please make the outside of the picture to be white since we'll be dealing with paper the outside must be white. Otherwise hanchel's function won't work.

So far I haven't enable OCR function yet. Only open image + crop + rotate

  • black white. Will try OCR tomorrow with merge from Hanchel + Andrew.

Hanchel, could you pleae tell me which functions do I call from you and which files do I need to compile to run your functions? Andrew, please finish it quick.

Quan.

On Sat, Mar 1, 2014 at 11:02 PM, Quan Quoc Minh Chau mqchau@uci.eduwrote:

Jamie, Your functions works ok with jpeg + png + gif + ppm. However, it DOESN'T work with bmp or pgm, maybe also pbm. For Alpha release I think it's OK for now. But for Beta release I want all those 3 formats to work as well. Quan.

On Sat, Mar 1, 2014 at 10:35 PM, Quan Quoc Minh Chau mqchau@uci.eduwrote:

Ok, thanks

On Sat, Mar 1, 2014 at 10:31 PM, jamiesl notifications@github.comwrote:

Okay, on my branch (ReadImage) I added Image.h and .c from the 22 solutions last quarter and added my function (IMAGE * ReadImage) to Model.h and Model.c. Everything from my machine should be pushed onto that branch already

Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/3#issuecomment-36447552 .

antrinh commented 10 years ago

Sorry, I am at dam gio. I will push and call you when I am home.

hanchelc commented 10 years ago

Quan, I gave the instructions for compiling my test above. I even included my makefile in my branch. make clean make TestIsolate ./TestIsolateCharacters

mqchau commented 10 years ago

Hanchel, I merged your parts with the rest and fixed two compatibility issues:

hanchelc commented 10 years ago

Nooooo What?? That's not the kind of result I saw the last time I ran it. D: I saw A and B very clearly.

Why don't you use Width and Height?

mqchau commented 10 years ago

Submitted everything. Merged to master. Everybody should branch from master to continue working or git checkout git rebase master

Close #3

Quan.

On Sun, Mar 2, 2014 at 4:23 PM, hanchelc notifications@github.com wrote:

Nooooo What?? That's not the kind of result I saw the last time I ran it. D: I saw A and B very clearly.

Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/3#issuecomment-36473254 .