Open mqchau opened 10 years ago
mqchau
commented
10 years ago Hanchel, please do the following git command to work on merged version of your code: git fetch git checkout HanchelToFixIsolate make TestIsolateCharacter bin/TestIsolateCharacter You only need to concern about src/Model/IsolateCharacter.c and src/Model/TestIsolateCharacter.c Also our new ReadImage function needs to pass in .ppm in the end of name. I fixed it in src/Model/TestIsolateCharacter.c
hanchelc
commented
10 years ago Quan, where the makefile? If I try to "make TestIsolate Character," I get gcc -c TestIsolateCharacter.c -I /home/group1/EECS22L/EECS22L_Project2/inc/Model In file included from /home/group1/EECS22L/EECS22L_Project2/inc/Model/Model.h:5:0, from TestIsolateCharacter.c:15: /home/group1/EECS22L/EECS22L_Project2/inc/Model/ModelStructure.h:5:21: fatal error: utarray.h: No such file or directory
^compilation terminated. make: *\ [TestIsolateCharacter.o] Error 1
mqchau
commented
10 years ago you have to make from the outside, at the EECS22L_Project2
On Sun, Mar 2, 2014 at 5:51 PM, hanchelc notifications@github.com wrote:
Quan, where the makefile? If I try to "make TestIsolate Character," I get gcc -c TestIsolateCharacter.c -I /home/group1/EECS22L/EECS22L_Project2/inc/Model In file included from /home/group1/EECS22L/EECS22L_Project2/inc/Model/Model.h:5:0, from TestIsolateCharacter.c:15: /home/group1/EECS22L/EECS22L_Project2/inc/Model/ModelStructure.h:5:21: fatal error: utarray.h: No such file or directory
include "utarray.h"
^ compilation terminated. make: *\ [TestIsolateCharacter.o] Error 1
Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-36475864 .
hanchelc
commented
10 years ago Team,
My Isolate Character has a little difficulty with " (quotation marks). It will spit back the quotation as two images in the image list, two ' s (single quotation mark or an apostrophe - however you want to identify it). SO, I propose that either Andrew or Kevin, whenever you see two apostrophes back to back, that you make it into a quotation mark.... I never see anybody use two apostrophes.. What do you guys think?
duongkt
commented
10 years ago hmmm, can't you make it so it grab at a certain width at least? that why it will include both ' ' into a character box?
for example if one ' is 5 width long, make it so it grab 10 width long,
On Fri, Mar 7, 2014 at 8:36 PM, hanchelc notifications@github.com wrote:
Team,
My Isolate Character has a little difficulty with " (quotation marks). It will spit back the quotation as two images in the image list, two ' s (single quotation mark or an apostrophe - however you want to identify it). SO, I propose that either Andrew or Kevin, whenever you see two apostrophes back to back, that you make it into a quotation mark.... I never see anybody use two apostrophes.. What do you guys think?
Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37088992 .
hanchelc
commented
10 years ago wtf? How am I gonna do it? If char == width of an apostrophe, grab double the amount? then how do I know I'm not grabbing the word "don't" or "can't"?? I think it would be easier for you to make the conditional of "if two apostrophes back to back, this is a quotation mark." than for me to assume this is a quotation mark and grab "double the width" of an apostrophe. Besides, how would I know the width of an apostrophe in each font without hardcoding some kind of dictionary or index for every apostrophe of every font size of every font.
duongkt
commented
10 years ago okie dokie. I will do that or unless andrew gonna include that in his part.
On Fri, Mar 7, 2014 at 8:47 PM, hanchelc notifications@github.com wrote:
wtf? How am I gonna do it? If char == width of an apostrophe, grab double the amount? then how do I know I'm not grabbing the word "don't" or "can't"?? I think it would be easier for you to make the conditional of "if two apostrophes back to back, this is a quotation mark." than for me to assume this is a quotation mark and grab "double the width" of an apostrophe. Besides, how would I know the width of an apostrophe in each font without hardcoding some kind of dictionary or index for every apostrophe of every font size of every font.
Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089143 .
mqchau
commented
10 years ago You can check for like 3 or 5 more columns on the right, if they're all white then this is single quotation mark, otherwise it is double quotation mark. I suggest that you leave this bug out for now. It won't affect the overall performance a lot.
Kevin, In your postPo, you can look for two consecutive apostrophe and cast that as a single quotation mark.
On Fri, Mar 7, 2014 at 8:39 PM, duongkt notifications@github.com wrote:
hmmm, can't you make it so it grab at a certain width at least? that why it will include both ' ' into a character box?
for example if one ' is 5 width long, make it so it grab 10 width long,
On Fri, Mar 7, 2014 at 8:36 PM, hanchelc notifications@github.com wrote:
Team,
My Isolate Character has a little difficulty with " (quotation marks). It will spit back the quotation as two images in the image list, two ' s (single quotation mark or an apostrophe - however you want to identify it). SO, I propose that either Andrew or Kevin, whenever you see two apostrophes back to back, that you make it into a quotation mark.... I never see anybody use two apostrophes.. What do you guys think?
Reply to this email directly or view it on GitHub< https://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37088992>
.
Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089025 .
hanchelc
commented
10 years ago how can I leave it out?! It's in every [include "header.h"] line!
mqchau
commented
10 years ago I mean that you don't worry about double quotation mark for now.
On Fri, Mar 7, 2014 at 8:55 PM, hanchelc notifications@github.com wrote:
how can I leave it out?! It's in every [include "header.h"] line!
Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089273 .
hanchelc
commented
10 years ago :+1:
Oh, btw, your logic from before has a little hitch. If 3 or 5 more columns on the right is white, then this is single quotation mark? Oh? Then what about words like "didn't" that to the right of the apostrophe, is another character. So I should make this a double quotation? It does work if the darker column on the right is identical do it, so that's where Kevin/Andrew could implement this condition. Aaaanway, glad we got this resolved. ^^v
mqchau
commented
10 years ago [?]
On Fri, Mar 7, 2014 at 8:58 PM, hanchelc notifications@github.com wrote:
[image: :+1:]
Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089324 .
hanchelc
commented
10 years ago What? It's a thumbs up. I don't think you can view it via email. It has to be on the github forum.
duongkt
commented
10 years ago i see it just fine in gmail lol
On Fri, Mar 7, 2014 at 9:01 PM, hanchelc notifications@github.com wrote:
What? It's a thumbs up. I don't think you can view it via email. It has to be on the github forum.
Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089394 .
hanchelc
commented
10 years ago Quan is confused about something....
mqchau
commented
10 years ago That' just a dance, not confusion.
On Fri, Mar 7, 2014 at 9:04 PM, hanchelc notifications@github.com wrote:
Quan is confused about something....
Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089438 .
antrinh
commented
10 years ago I think the detecting of 2 apostrophes in a row would have to be part of kevin's because I'm not sure where I would put such a thing in my code since I'm returning the probability of a match for each character
On Fri, Mar 7, 2014 at 9:08 PM, mqchau notifications@github.com wrote:
That' just a dance, not confusion.
On Fri, Mar 7, 2014 at 9:04 PM, hanchelc notifications@github.com wrote:
Quan is confused about something....
Reply to this email directly or view it on GitHub< https://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089438>
.
Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089507 .
antrinh
commented
10 years ago Hey all, just got home. I know I didn't deliver from the alpha release, and I thank you guys for covering for my shortcomings in that submission. I will have a fully functional IDchar for the Beta release, and ideally more for the final.
On Sat, Mar 8, 2014 at 11:26 AM, Andrew Nam Trinh antrinh@uci.edu wrote:
I think the detecting of 2 apostrophes in a row would have to be part of kevin's because I'm not sure where I would put such a thing in my code since I'm returning the probability of a match for each character
On Fri, Mar 7, 2014 at 9:08 PM, mqchau notifications@github.com wrote:
That' just a dance, not confusion.
On Fri, Mar 7, 2014 at 9:04 PM, hanchelc notifications@github.com wrote:
Quan is confused about something....
Reply to this email directly or view it on GitHub< https://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089438
.
Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089507 .
antrinh
commented
10 years ago Alright, just finished a session of working on the code. I have pushed to the branch "IdentifyCharacter4". Here are the current issues:
If someone would like to look at the code, then feel free to do so, but I will be back to working on the code at around 2-3pm. Quan, can I come work at your house around that time?
On Sat, Mar 8, 2014 at 11:17 PM, Andrew Nam Trinh antrinh@uci.edu wrote:
Hey all, just got home. I know I didn't deliver from the alpha release, and I thank you guys for covering for my shortcomings in that submission. I will have a fully functional IDchar for the Beta release, and ideally more for the final.
On Sat, Mar 8, 2014 at 11:26 AM, Andrew Nam Trinh antrinh@uci.edu wrote:
I think the detecting of 2 apostrophes in a row would have to be part of kevin's because I'm not sure where I would put such a thing in my code since I'm returning the probability of a match for each character
On Fri, Mar 7, 2014 at 9:08 PM, mqchau notifications@github.com wrote:
That' just a dance, not confusion.
On Fri, Mar 7, 2014 at 9:04 PM, hanchelc notifications@github.com wrote:
Quan is confused about something....
Reply to this email directly or view it on GitHub< https://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089438
.
Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089507 .
mqchau
commented
10 years ago Im ok with that. Text me b4 u come. Btw, could jamie look at his code between now andvafternoon? On Mar 9, 2014 6:47 AM, "Andrew Trinh" notifications@github.com wrote:
Alright, just finished a session of working on the code. I have pushed to the branch "IdentifyCharacter4". Here are the current issues:
- Seg fault while compiling.
- Cygwin didn't have gdb so I couldn't debug.
- I would install gdb, but I have to be somewhere from 7:30am to 2-3pm (not exact).
If someone would like to look at the code, then feel free to do so, but I will be back to working on the code at around 2-3pm. Quan, can I come work at your house around that time?
On Sat, Mar 8, 2014 at 11:17 PM, Andrew Nam Trinh antrinh@uci.edu wrote:
Hey all, just got home. I know I didn't deliver from the alpha release, and I thank you guys for covering for my shortcomings in that submission. I will have a fully functional IDchar for the Beta release, and ideally more for the final.
On Sat, Mar 8, 2014 at 11:26 AM, Andrew Nam Trinh antrinh@uci.edu wrote:
I think the detecting of 2 apostrophes in a row would have to be part of kevin's because I'm not sure where I would put such a thing in my code since I'm returning the probability of a match for each character
On Fri, Mar 7, 2014 at 9:08 PM, mqchau notifications@github.com wrote:
That' just a dance, not confusion.
On Fri, Mar 7, 2014 at 9:04 PM, hanchelc notifications@github.com wrote:
Quan is confused about something....
Reply to this email directly or view it on GitHub<
https://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089438
.
Reply to this email directly or view it on GitHub< https://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37089507> .
Reply to this email directly or view it on GitHubhttps://github.com/mqchau/EECS22L_Project2/issues/4#issuecomment-37127275 .
For beta release, this is an incomplete action items we need to accomplish:
By the way, for about page, I want everybody to be at the group meeting at 2pm on Tue in the lab so we can take a group picture and put it in our program. If some of you can't attend we'll manage by taking individual pictures.
Quan.