-
Notifications
You must be signed in to change notification settings - Fork 50
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
following the program is not an easy thing #6
Comments
@mnfsoft Hello, I'm sorry for that. The code it's pretty old but I will try to clean and comment it. 😄 |
Dear Vision Race Code Owner (I actually did not know how to address you)
Thank you very much for your swift reply!
I can help you on the issues that I did not quiet understand
1. *from Utils import * *
I mean it is only 4 defenitions they are simple would there be a problem if
you move them to Main or there would be a problem on RPi if it is unified?!
2. *is there any reason why you have used the time function?*
time.clock() in the Main.py file especially you did not use it (i.e. t2-t1)
I have used it but to measure time at two instances then I used the
expression t2-t1 to measure the user Replytime
3. *fm = RepackImages(N_Clice) this function needs more comments*
this function is located in the file Utils.py
4. *Images = [] can we define an empty array???*
*5. Images.append(Image())*
Append is it an external command or an internal command or method you have
created because I really do not know what it means ? or what it does?
I hope after these the code would look better and be much more
readable!...or be esily readable!
Because the main intention of whoever made PYTHON was the simplicity and
the code would
yours truly,
Mohammed Al-Alaw
…On Thu, Apr 26, 2018 at 2:21 PM, cTatu ***@***.***> wrote:
@mnfsoft <https://github.com/mnfsoft> Hello, I'm sorry for that. The code
it's pretty old but I will try to clean and comment it. 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AfgWC8mORMivuutVaP8Jw14zKRvD_x_Rks5tsa3IgaJpZM4TjhUz>
.
|
plus whoever teaches coding should also give brief notes on software
engineering or towards easily readable codes
yours truly,
Mohammed Al-Alaw
…On Thu, Apr 26, 2018 at 5:46 PM, Mohammed Ahmed ***@***.***> wrote:
Dear Vision Race Code Owner (I actually did not know how to address you)
Thank you very much for your swift reply!
I can help you on the issues that I did not quiet understand
1. *from Utils import * *
I mean it is only 4 defenitions they are simple would there be a problem
if you move them to Main or there would be a problem on RPi if it is
unified?!
2. *is there any reason why you have used the time function?*
time.clock() in the Main.py file especially you did not use it (i.e. t2-t1)
I have used it but to measure time at two instances then I used the
expression t2-t1 to measure the user Replytime
3. *fm = RepackImages(N_Clice) this function needs more comments*
this function is located in the file Utils.py
4. *Images = [] can we define an empty array???*
*5. Images.append(Image())*
Append is it an external command or an internal command or method you have
created because I really do not know what it means ? or what it does?
I hope after these the code would look better and be much more
readable!...or be esily readable!
Because the main intention of whoever made PYTHON was the simplicity and
the code would
yours truly,
Mohammed Al-Alaw
On Thu, Apr 26, 2018 at 2:21 PM, cTatu ***@***.***> wrote:
> @mnfsoft <https://github.com/mnfsoft> Hello, I'm sorry for that. The
> code it's pretty old but I will try to clean and comment it. 😄
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#6 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AfgWC8mORMivuutVaP8Jw14zKRvD_x_Rks5tsa3IgaJpZM4TjhUz>
> .
>
|
Hello, I will try to answer your question.
1. The Utils file only handles 4 functions to process the images. If you want you can move them to the main file, that will not affect anything.
2. I only use the time library to measure how much expensive is the proccesing task of the images because the code is ment for a Raspberry Pi Zero.
3. The function RepackImages recieve an array of images (previously sliced with the SlicePart function) and concatenate them. If you have 4 slices this functions will return 1 image made of all those 4 slices.
4. Yes, it is like defining an empty array in C or Java.
5. The function append it’s a function defined in the Python Language, like the definition of an empty array (4.). Adds an object to a list an element, which in this case is a class of type Image
I hope it helps, best regards!
De: mnfsoft
Enviado: jueves, 26 de abril de 2018 16:46
Para: CRM-UAM/VisionRace
CC: cTatu; Comment
Asunto: Re: [CRM-UAM/VisionRace] following the program is not an easy thing(#6)
Dear Vision Race Code Owner (I actually did not know how to address you)
Thank you very much for your swift reply!
I can help you on the issues that I did not quiet understand
1. *from Utils import * *
I mean it is only 4 defenitions they are simple would there be a problem if
you move them to Main or there would be a problem on RPi if it is unified?!
2. *is there any reason why you have used the time function?*
time.clock() in the Main.py file especially you did not use it (i.e. t2-t1)
I have used it but to measure time at two instances then I used the
expression t2-t1 to measure the user Replytime
3. *fm = RepackImages(N_Clice) this function needs more comments*
this function is located in the file Utils.py
4. *Images = [] can we define an empty array???*
*5. Images.append(Image())*
Append is it an external command or an internal command or method you have
created because I really do not know what it means ? or what it does?
I hope after these the code would look better and be much more
readable!...or be esily readable!
Because the main intention of whoever made PYTHON was the simplicity and
the code would
yours truly,
Mohammed Al-Alaw
On Thu, Apr 26, 2018 at 2:21 PM, cTatu ***@***.***> wrote:
@mnfsoft <https://github.com/mnfsoft> Hello, I'm sorry for that. The code
it's pretty old but I will try to clean and comment it. 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AfgWC8mORMivuutVaP8Jw14zKRvD_x_Rks5tsa3IgaJpZM4TjhUz>
.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
|
Dear Developer,
For the record traditional C language does not allow an empty array you
must define it via 'memory allocation' I mean it is no easy job because C
lang. was created for the memory efficiency!; I mean it was created in a
time where memory was expensive!.
thank you very much!
yours truly,
Mohammed Al-Alaw
…On Fri, Apr 27, 2018 at 9:25 PM, cTatu ***@***.***> wrote:
Hello, I will try to answer your question.
1. The Utils file only handles 4 functions to process the images. If you
want you can move them to the main file, that will not affect anything.
2. I only use the time library to measure how much expensive is the
proccesing task of the images because the code is ment for a Raspberry Pi
Zero.
3. The function RepackImages recieve an array of images (previously sliced
with the SlicePart function) and concatenate them. If you have 4 slices
this functions will return 1 image made of all those 4 slices.
4. Yes, it is like defining an empty array in C or Java.
5. The function append it’s a function defined in the Python Language,
like the definition of an empty array (4.). Adds an object to a list an
element, which in this case is a class of type Image
I hope it helps, best regards!
De: mnfsoft
Enviado: jueves, 26 de abril de 2018 16:46
Para: CRM-UAM/VisionRace
CC: cTatu; Comment
Asunto: Re: [CRM-UAM/VisionRace] following the program is not an easy
thing(#6)
Dear Vision Race Code Owner (I actually did not know how to address you)
Thank you very much for your swift reply!
I can help you on the issues that I did not quiet understand
1. *from Utils import * *
I mean it is only 4 defenitions they are simple would there be a problem if
you move them to Main or there would be a problem on RPi if it is unified?!
2. *is there any reason why you have used the time function?*
time.clock() in the Main.py file especially you did not use it (i.e. t2-t1)
I have used it but to measure time at two instances then I used the
expression t2-t1 to measure the user Replytime
3. *fm = RepackImages(N_Clice) this function needs more comments*
this function is located in the file Utils.py
4. *Images = [] can we define an empty array???*
*5. Images.append(Image())*
Append is it an external command or an internal command or method you have
created because I really do not know what it means ? or what it does?
I hope after these the code would look better and be much more
readable!...or be esily readable!
Because the main intention of whoever made PYTHON was the simplicity and
the code would
yours truly,
Mohammed Al-Alaw
On Thu, Apr 26, 2018 at 2:21 PM, cTatu ***@***.***> wrote:
> @mnfsoft <https://github.com/mnfsoft> Hello, I'm sorry for that. The
code
> it's pretty old but I will try to clean and comment it. 😄
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#6 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/
AfgWC8mORMivuutVaP8Jw14zKRvD_x_Rks5tsa3IgaJpZM4TjhUz>
> .
>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AfgWCxY1JE0g7_6Qkz98WC-Qo69C-7Wwks5ts2KIgaJpZM4TjhUz>
.
|
Sorry y delete the last I meant to say:
Dear Developer,
… For the record traditional C language does not allow an empty array
; but if you want to do this,
you must define it via 'memory allocation' I mean it is no easy job
because C lang. was created for the memory efficiency!; I mean it was
created in a time where memory was expensive!.
thank you very much!
yours truly,
Mohammed Al-Alaw
On Fri, Apr 27, 2018 at 9:25 PM, cTatu ***@***.***> wrote:
>
> Hello, I will try to answer your question.
>
> 1. The Utils file only handles 4 functions to process the images. If you
> want you can move them to the main file, that will not affect anything.
> 2. I only use the time library to measure how much expensive is the
> proccesing task of the images because the code is ment for a Raspberry Pi
> Zero.
> 3. The function RepackImages recieve an array of images (previously
> sliced with the SlicePart function) and concatenate them. If you have 4
> slices this functions will return 1 image made of all those 4 slices.
> 4. Yes, it is like defining an empty array in C or Java.
> 5. The function append it’s a function defined in the Python Language,
> like the definition of an empty array (4.). Adds an object to a list an
> element, which in this case is a class of type Image
>
> I hope it helps, best regards!
>
> De: mnfsoft
> Enviado: jueves, 26 de abril de 2018 16:46
> Para: CRM-UAM/VisionRace
> CC: cTatu; Comment
> Asunto: Re: [CRM-UAM/VisionRace] following the program is not an easy
> thing(#6)
>
> Dear Vision Race Code Owner (I actually did not know how to address you)
> Thank you very much for your swift reply!
> I can help you on the issues that I did not quiet understand
>
> 1. *from Utils import * *
>
> I mean it is only 4 defenitions they are simple would there be a problem
> if
> you move them to Main or there would be a problem on RPi if it is
> unified?!
>
>
> 2. *is there any reason why you have used the time function?*
> time.clock() in the Main.py file especially you did not use it (i.e.
> t2-t1)
> I have used it but to measure time at two instances then I used the
> expression t2-t1 to measure the user Replytime
>
> 3. *fm = RepackImages(N_Clice) this function needs more comments*
> this function is located in the file Utils.py
>
> 4. *Images = [] can we define an empty array???*
>
> *5. Images.append(Image())*
> Append is it an external command or an internal command or method you have
> created because I really do not know what it means ? or what it does?
>
> I hope after these the code would look better and be much more
> readable!...or be esily readable!
>
> Because the main intention of whoever made PYTHON was the simplicity and
> the code would
>
> yours truly,
> Mohammed Al-Alaw
>
>
> On Thu, Apr 26, 2018 at 2:21 PM, cTatu ***@***.***> wrote:
>
> > @mnfsoft <https://github.com/mnfsoft> Hello, I'm sorry for that. The
> code
> > it's pretty old but I will try to clean and comment it. 😄
> >
> > —
> > You are receiving this because you were mentioned.
> > Reply to this email directly, view it on GitHub
> > <#6 (comment)
> >,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-auth/AfgWC8mOR
> MivuutVaP8Jw14zKRvD_x_Rks5tsa3IgaJpZM4TjhUz>
> > .
> >
>
> —
> You are receiving this because you commented.
>
> Reply to this email directly, view it on GitHub, or mute the thread.
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#6 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AfgWCxY1JE0g7_6Qkz98WC-Qo69C-7Wwks5ts2KIgaJpZM4TjhUz>
> .
>
|
Dear Developer ;
I also checked with youtube it is also not that direct in JAVA
it is not that direct
yours truly,
Mohammed Al-Alaw
…On Thu, May 3, 2018 at 4:31 PM, Mohammed Ahmed ***@***.***> wrote:
Sorry y delete the last I meant to say:
Dear Developer,
> For the record traditional C language does not allow an empty array
> ; but if you want to do this,
> you must define it via 'memory allocation' I mean it is no easy job
> because C lang. was created for the memory efficiency!; I mean it was
> created in a time where memory was expensive!.
>
> thank you very much!
>
> yours truly,
> Mohammed Al-Alaw
>
>
> On Fri, Apr 27, 2018 at 9:25 PM, cTatu ***@***.***> wrote:
>
>>
>> Hello, I will try to answer your question.
>>
>> 1. The Utils file only handles 4 functions to process the images. If you
>> want you can move them to the main file, that will not affect anything.
>> 2. I only use the time library to measure how much expensive is the
>> proccesing task of the images because the code is ment for a Raspberry Pi
>> Zero.
>> 3. The function RepackImages recieve an array of images (previously
>> sliced with the SlicePart function) and concatenate them. If you have 4
>> slices this functions will return 1 image made of all those 4 slices.
>> 4. Yes, it is like defining an empty array in C or Java.
>> 5. The function append it’s a function defined in the Python Language,
>> like the definition of an empty array (4.). Adds an object to a list an
>> element, which in this case is a class of type Image
>>
>> I hope it helps, best regards!
>>
>> De: mnfsoft
>> Enviado: jueves, 26 de abril de 2018 16:46
>> Para: CRM-UAM/VisionRace
>> CC: cTatu; Comment
>> Asunto: Re: [CRM-UAM/VisionRace] following the program is not an easy
>> thing(#6)
>>
>> Dear Vision Race Code Owner (I actually did not know how to address you)
>> Thank you very much for your swift reply!
>> I can help you on the issues that I did not quiet understand
>>
>> 1. *from Utils import * *
>>
>> I mean it is only 4 defenitions they are simple would there be a problem
>> if
>> you move them to Main or there would be a problem on RPi if it is
>> unified?!
>>
>>
>> 2. *is there any reason why you have used the time function?*
>> time.clock() in the Main.py file especially you did not use it (i.e.
>> t2-t1)
>> I have used it but to measure time at two instances then I used the
>> expression t2-t1 to measure the user Replytime
>>
>> 3. *fm = RepackImages(N_Clice) this function needs more comments*
>> this function is located in the file Utils.py
>>
>> 4. *Images = [] can we define an empty array???*
>>
>> *5. Images.append(Image())*
>> Append is it an external command or an internal command or method you
>> have
>> created because I really do not know what it means ? or what it does?
>>
>> I hope after these the code would look better and be much more
>> readable!...or be esily readable!
>>
>> Because the main intention of whoever made PYTHON was the simplicity and
>> the code would
>>
>> yours truly,
>> Mohammed Al-Alaw
>>
>>
>> On Thu, Apr 26, 2018 at 2:21 PM, cTatu ***@***.***> wrote:
>>
>> > @mnfsoft <https://github.com/mnfsoft> Hello, I'm sorry for that. The
>> code
>> > it's pretty old but I will try to clean and comment it. 😄
>> >
>> > —
>> > You are receiving this because you were mentioned.
>> > Reply to this email directly, view it on GitHub
>> > <#6 (comment)
>> >,
>> > or mute the thread
>> > <https://github.com/notifications/unsubscribe-auth/AfgWC8mOR
>> MivuutVaP8Jw14zKRvD_x_Rks5tsa3IgaJpZM4TjhUz>
>> > .
>> >
>>
>> —
>> You are receiving this because you commented.
>>
>> Reply to this email directly, view it on GitHub, or mute the thread.
>>
>> —
>> You are receiving this because you were mentioned.
>> Reply to this email directly, view it on GitHub
>> <#6 (comment)>,
>> or mute the thread
>> <https://github.com/notifications/unsubscribe-auth/AfgWCxY1JE0g7_6Qkz98WC-Qo69C-7Wwks5ts2KIgaJpZM4TjhUz>
>> .
>>
>
>
|
Hi! Thanks to your python algorithm of line detection and direction calculation I've finally got my balancing robot to follow the line for my masters diploma :) you can check it here if you like https://www.instagram.com/nickzherdev/ |
Dear Nick,
Sorry for the late reply;
well I find the first algorithm very useful although you call it slow but I
cannot find the command that connects the centroids and draws them dark
blue! plus why did you write the line:
N_SLICES = 4
where I do not understand its role?
yours truly,
Mohammed Al-Alaw
…On Tue, May 8, 2018 at 3:58 PM, Nick ***@***.***> wrote:
Hi! Thanks to your python algorithm of line detection and direction
calculation I've finally got my balancing robot to follow the line for my
masters diploma :) you can check it here if you like
https://www.instagram.com/nickzherdev/
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AfgWCze6YHeJjTp3zEY_sZvfHlJ-qSPtks5twZaNgaJpZM4TjhUz>
.
|
Dear Developer,
I really read the code for lets say at least twice but I can not find the
line that connects the centroids!
yours truly,
Mohammed Al-Alaw
…On Mon, May 14, 2018 at 4:52 PM, Mohammed Ahmed ***@***.***> wrote:
Dear Nick,
Sorry for the late reply;
well I find the first algorithm very useful although you call it slow but
I cannot find the command that connects the centroids and draws them dark
blue! plus why did you write the line:
N_SLICES = 4
where I do not understand its role?
yours truly,
Mohammed Al-Alaw
On Tue, May 8, 2018 at 3:58 PM, Nick ***@***.***> wrote:
> Hi! Thanks to your python algorithm of line detection and direction
> calculation I've finally got my balancing robot to follow the line for my
> masters diploma :) you can check it here if you like
> https://www.instagram.com/nickzherdev/
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#6 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/AfgWCze6YHeJjTp3zEY_sZvfHlJ-qSPtks5twZaNgaJpZM4TjhUz>
> .
>
|
Dear Developer,
I really read the code for lets say at least twice but I can not find the
line that connects the centroids!
P.S. my Whats up no. is +905537658971
yours truly,
Mohammed Al-Alaw
…On Fri, Apr 27, 2018 at 9:25 PM, cTatu ***@***.***> wrote:
Hello, I will try to answer your question.
1. The Utils file only handles 4 functions to process the images. If you
want you can move them to the main file, that will not affect anything.
2. I only use the time library to measure how much expensive is the
proccesing task of the images because the code is ment for a Raspberry Pi
Zero.
3. The function RepackImages recieve an array of images (previously sliced
with the SlicePart function) and concatenate them. If you have 4 slices
this functions will return 1 image made of all those 4 slices.
4. Yes, it is like defining an empty array in C or Java.
5. The function append it’s a function defined in the Python Language,
like the definition of an empty array (4.). Adds an object to a list an
element, which in this case is a class of type Image
I hope it helps, best regards!
De: mnfsoft
Enviado: jueves, 26 de abril de 2018 16:46
Para: CRM-UAM/VisionRace
CC: cTatu; Comment
Asunto: Re: [CRM-UAM/VisionRace] following the program is not an easy
thing(#6)
Dear Vision Race Code Owner (I actually did not know how to address you)
Thank you very much for your swift reply!
I can help you on the issues that I did not quiet understand
1. *from Utils import * *
I mean it is only 4 defenitions they are simple would there be a problem if
you move them to Main or there would be a problem on RPi if it is unified?!
2. *is there any reason why you have used the time function?*
time.clock() in the Main.py file especially you did not use it (i.e. t2-t1)
I have used it but to measure time at two instances then I used the
expression t2-t1 to measure the user Replytime
3. *fm = RepackImages(N_Clice) this function needs more comments*
this function is located in the file Utils.py
4. *Images = [] can we define an empty array???*
*5. Images.append(Image())*
Append is it an external command or an internal command or method you have
created because I really do not know what it means ? or what it does?
I hope after these the code would look better and be much more
readable!...or be esily readable!
Because the main intention of whoever made PYTHON was the simplicity and
the code would
yours truly,
Mohammed Al-Alaw
On Thu, Apr 26, 2018 at 2:21 PM, cTatu ***@***.***> wrote:
> @mnfsoft <https://github.com/mnfsoft> Hello, I'm sorry for that. The
code
> it's pretty old but I will try to clean and comment it. 😄
>
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub
> <#6 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/
AfgWC8mORMivuutVaP8Jw14zKRvD_x_Rks5tsa3IgaJpZM4TjhUz>
> .
>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AfgWCxY1JE0g7_6Qkz98WC-Qo69C-7Wwks5ts2KIgaJpZM4TjhUz>
.
|
Be careful - you just shared your phone number with the whole internet! ;) |
Dear Developer,
0.k. I know this!
but I badly need to know where is the command line that you connected the
dots after that it is as easy as using the equation
y2 - y1
------------------- = tan (theta)
x1-x2
where theta is the angle between two lines where I decided that
if theta = or larger than 90 degrees or pi/2 we should issue the command to
go right or activate the right motor
if theta = or smaller than 90 degrees or pi/2 activate the left motor
it needs fine tuning when we operate in practical but that is the
methodology of operation in general!
but I really need to analyse the code in general!
yours truly,
Mohammed Al-Alaw
…On Wed, May 30, 2018 at 8:15 PM, Carlos Garcia Saura < ***@***.***> wrote:
Be careful - you just shared your phone number with the whole internet! ;)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#6 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AfgWC1429EIyc1D46nP9IbAuNEG-idIhks5t3tPHgaJpZM4TjhUz>
.
|
Dear Sir,
I have spent more than a week trying to untangle your spaghetti code and I must admit that you have many declarations and re-declarations that are hard to understand!
a little more comments would be favorable plus if you may reduce the unnecessary re-declarations that would ease our life!
The text was updated successfully, but these errors were encountered: