Page 12 of 14

Re: CPUID - характеристики центрального процессора

Posted: Fri Aug 19, 2016 11:55 pm
by Wildwest
Wildwest wrote:Also, we need a new exception message Unknown model (maybe Unknown Codename)
If it doesn't go until this jump (so the model is know) app works fine.

But checking case, where it should jump to label, crushes app.

Re: CPUID - характеристики центрального процессора

Posted: Wed Aug 24, 2016 10:26 pm
by Wildwest
I asked mat1854 to put new version with full sources on forum in 1-2 days (since her SVN commits were rejected), so everyone can test it.

Re: CPUID - характеристики центрального процессора

Posted: Thu Aug 25, 2016 5:29 am
by mat1854
Here is the latest version. which compiles fine as a stand alone app. But for the SVN commit these file paths have changed

These two :
include 'load_lib.mac'
include 'macros.inc' ; useful macroses

to :
include '..\..\..\develop\libraries\box_lib\load_lib.mac'
include '..\..\..\macros.inc'

Can someone help me in committing this file? Only CPUID.ASM has changed since last commit.

Re: CPUID - характеристики центрального процессора

Posted: Thu Aug 25, 2016 8:45 am
by Wildwest
Not now, we will test it before commit and the version would be changed (maybe 2.45).

And like in June viewtopic.php?f=42&t=594&start=135#p65707 write answers

Write here a list of things, which were done by you after midterm.
Write here a list of things, which weren't done by you after midterm, including reasons, why were not done yet.

Re: CPUID - характеристики центрального процессора

Posted: Sat Aug 27, 2016 6:41 am
by mat1854
Fixed bugs. Here is the report where everything is in detail https://docs.google.com/document/d/1qOl ... sp=sharing
Added "unknown codename" message.
Disabled multiplier detection for latest AMD.
Fixed detection of AMD Phenom II.

Known issue: MENUET01 string appears when the application first loads. It disappears after any window movement.

Note: I do not have internet service where I am staying. Hope this helps out.

Re: CPUID - характеристики центрального процессора

Posted: Sat Aug 27, 2016 2:07 pm
by Wildwest
New tests

h1.jpg - just start of application in case, if it know model. Strange things, maybe redraw bug + something new... Artefacts disappears, if you click at Press for more button or move window .

h2.jpg - crush of application in case, if it don't know model.

Re: CPUID - характеристики центрального процессора

Posted: Sat Aug 27, 2016 7:55 pm
by dunkaist
Not much changes since last try.

Re: CPUID - характеристики центрального процессора

Posted: Sun Aug 28, 2016 4:33 pm
by Wildwest
Yes, but at least no false detection of multiplier and clock.

Re: CPUID - характеристики центрального процессора

Posted: Fri Sep 09, 2016 4:10 pm
by mat1854
I do not want the code I have written after the midterm to be used in the CPUID application. This means all the listed things I have posted earlier.

Re: CPUID - характеристики центрального процессора

Posted: Fri Sep 09, 2016 4:55 pm
by hidnplayr
mat1854 wrote:I do not want the code I have written after the midterm to be used in the CPUID application. This means all the listed things I have posted earlier.
I'm sorry to hear that.
You do realize this way you'll be violating the terms you agreed with while entering the program, and what consequences this could have?

Re: CPUID - характеристики центрального процессора

Posted: Fri Sep 09, 2016 5:36 pm
by ashmew2
Do we need to rewrite everything after mid term then?
I don't think violating any licenses (no matter how informal) should be practiced...

Re: CPUID - характеристики центрального процессора

Posted: Fri Sep 09, 2016 5:39 pm
by ashmew2
mat1854 wrote:I do not want the code I have written after the midterm to be used in the CPUID application. This means all the listed things I have posted earlier.
Could you clarify what you mean?
Are you saying this because you feel the quality of code does not match your expectations of yourself?
OR
Do you mean we should not use it because you (being the author) do not want it to be used in CPUID?

Re: CPUID - характеристики центрального процессора

Posted: Fri Sep 09, 2016 7:01 pm
by Wildwest
https://summerofcode.withgoogle.com/terms/student Your lose any rights on this code

Re: CPUID - характеристики центрального процессора

Posted: Fri Sep 09, 2016 8:32 pm
by Pathoswithin
mat1854
Not mentioning the rules, you have received pass on midterm evaluation by me in advance, while Wildwest was not completely agree, but on final evaluation he was confident. Summary result is about 50%, so half money is sort of fare.

Re: CPUID - характеристики центрального процессора

Posted: Sun Sep 11, 2016 12:17 am
by mat1854
Wildwest asked me to write the code after the actual deadline because Pathoswithin asked him about something. So many problems with his mentorship. But if we can not change my fail status now, all I want to make clear is I DO NOT GIVE PERMISSION TO USE THE CODE I HAVE WRITTEN AFTER THE MIDTERM.

If we can still change the status ,I can give explanation.

Thanks.