UICHCC / UIC_GPA_Calculator

A GPA calculator for UICer.
https://uichcc.com/UIC_GPA_Calculator/
1 stars 0 forks source link

Review of Algorithm #1

Open jyhi opened 6 years ago

jyhi commented 6 years ago

In previous discussion someone (maybe @ecwu ? I can't remember) has raised consideration on the algorithm's generality: different department(programme) may have different semester goals, leading to different algorithm parameters.

Current algorithm is designed for Computer Science and Technology (CST). A review need to be taken to guarantee the accuracy of the calculator.

ecwu commented 6 years ago
  Y1A Y1B Y2A Y2B Y3A Y3B Y4A Y4B
DBM 17 17 19 19 18 18 18 6
CTV 17 17 19 19 18 18 18 6
CCM/MAD 17 20 19 19 18 18 15 6
MOST DHSS 17 17 19 19 18 18 18 6
IJ 17 17 18 20 19 16 17 8
SWSA 17 17 19 19 18 21 15 6
MOST DST 17 17 19 19 18 18 18 6
ENVS STAT APSY 17 17 19 19 18 18 18 6
CST 17 18 19 20 18 19 15 6
DS 17 17 19 20 19 19 15 6
FM 20 20 19 16 18 18 15 6
FST 17 20 19 19 18 18 15 6

I make this table according to Undergraduate Handbook 2017-2018.

We need to design a new algorithm...

billzhonggz commented 6 years ago

I suggest follows:

  1. Add data of 2014-2016 & 2018 admissions.
  2. The calculator can receive HTTP GET with parameters to show a result directly. It can make the result easy to be distributed. e.g. https://gpacal.uichcc.app/?admission=2017&major=CST&current=2A&offest=+3&dest=4B

The value of the parameter offset is added to the preset credit to indicate any abnormal situations, like exchange programs.

jyhi commented 6 years ago

The calculator can receive HTTP GET with parameters to show a result directly.

@ecwu what do you think about adding a "backend"?

e.g. https://gpacal.uichcc.app/?admission=2017&major=CST&current=2A&offest=+3&dest=4B

s/offest/offset/g

It might be better to mount API on a differnet path, e.g.:

https://gpacal.uichcc.app/api?admission=2017&major=CST&current=2A&offest=+3&dest=4B

(I tried REST and it's simply not elegant in this case)

ecwu commented 6 years ago

@lmy441900 I think @billzhonggz 's idea will work. In addition, we can add a new function (set a little goal for you next semester) for more interaction and fun.

billzhonggz commented 6 years ago

I don't think we need a back end. JavaScript can do this.

Get Outlook for Androidhttps://aka.ms/ghei36


From: Junde Yhi notifications@github.com Sent: Monday, July 9, 2018 11:36:38 PM To: UICHCC/UIC_GPA_Calculator Cc: Bill ZHONG; Comment Subject: Re: [UICHCC/UIC_GPA_Calculator] Review of Algorithm (#1)

The calculator can receive HTTP GET with parameters to show a result directly.

@ecwuhttps://github.com/ecwu what do you think about adding a "backend"?

e.g. https://gpacal.uichcc.app/?admission=2017&major=CST&current=2A&offest=+3&dest=4B

s/offest/offset/g

It might be better to mount API on a differnet path, e.g.:

https://gpacal.uichcc.app/api?admission=2017&major=CST&current=2A&offest=+3&dest=4B

(I tried REST and it's simply not elegant in this case)

— You are receiving this because you commented. Reply to this email directly, view it on GitHubhttps://github.com/UICHCC/UIC_GPA_Calculator/issues/1#issuecomment-403522044, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMevzLkEo2AOslhUv8SrQx040__UyTKAks5uE3iGgaJpZM4VGq4i.

jyhi commented 6 years ago

Do you mean the app itself can act as a server?

ecwu commented 6 years ago

@lmy441900 HTTP GET provides the data, JS process it.

billzhonggz commented 6 years ago

JS can take these parameters and do calculation automatically. I have seen this feature in Vue.

Get Outlook for Androidhttps://aka.ms/ghei36


From: Junde Yhi notifications@github.com Sent: Monday, July 9, 2018 11:44:58 PM To: UICHCC/UIC_GPA_Calculator Cc: Bill ZHONG; Mention Subject: Re: [UICHCC/UIC_GPA_Calculator] Review of Algorithm (#1)

Do you mean the app itself can act as a server?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/UICHCC/UIC_GPA_Calculator/issues/1#issuecomment-403524906, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMevzLe39pPkn6kcgCGUbBp5csW6q0WCks5uE3p6gaJpZM4VGq4i.

jyhi commented 6 years ago

Alright, I know you guys... Basically providing the calculator parameters.

What is the actual use case of this functionality then? Normally people use API for data only, not the web page.

ecwu commented 6 years ago

@lmy441900 Maybe api is not the correct way to name it.

billzhonggz commented 6 years ago

Weixin sharing, maybe. By having the link, a same result can be produced. I don't think this is an API, this is not intended to provide any service to third-parties. This idea is designed to make sharing easier.

Get Outlook for Androidhttps://aka.ms/ghei36


From: Junde Yhi notifications@github.com Sent: Monday, July 9, 2018 11:48:44 PM To: UICHCC/UIC_GPA_Calculator Cc: Bill ZHONG; Mention Subject: Re: [UICHCC/UIC_GPA_Calculator] Review of Algorithm (#1)

Alright, I know you guys... Basically providing the calculator parameters.

What is the actual use case of this functionality then? Normally people use API for data only, not the web page.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/UICHCC/UIC_GPA_Calculator/issues/1#issuecomment-403526159, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMevzLWcqaM7OJ0mkHhMYede9hl-J_kHks5uE3tcgaJpZM4VGq4i.

jyhi commented 6 years ago

@billzhonggz Alright then.

ecwu commented 6 years ago

@billzhonggz Can you explain the offset parameter?

billzhonggz commented 6 years ago

Offset is added to the preset credit base on handbooks to indicate any transferred or dropped credits. It can be both positive and negative, and its default value is 0.

Get Outlook for Androidhttps://aka.ms/ghei36


From: Jack Wu notifications@github.com Sent: Tuesday, July 10, 2018 12:05:49 AM To: UICHCC/UIC_GPA_Calculator Cc: Bill ZHONG; Mention Subject: Re: [UICHCC/UIC_GPA_Calculator] Review of Algorithm (#1)

@billzhonggzhttps://github.com/billzhonggz Can you explain the offset parameter?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/UICHCC/UIC_GPA_Calculator/issues/1#issuecomment-403531774, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AMevzFA7WuIEkvkCgwwySbSalBBYoD2Aks5uE39dgaJpZM4VGq4i.