Skip to content
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

Review of Algorithm #1

Open
jyhi opened this issue Jul 8, 2018 · 14 comments
Open

Review of Algorithm #1

jyhi opened this issue Jul 8, 2018 · 14 comments

Comments

@jyhi
Copy link
Member

jyhi commented Jul 8, 2018

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
Copy link
Member

ecwu commented Jul 8, 2018

  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
Copy link
Member

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
Copy link
Member Author

jyhi commented Jul 9, 2018

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
Copy link
Member

ecwu commented Jul 9, 2018

@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
Copy link
Member

billzhonggz commented Jul 9, 2018 via email

@jyhi
Copy link
Member Author

jyhi commented Jul 9, 2018

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

@ecwu
Copy link
Member

ecwu commented Jul 9, 2018

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

@billzhonggz
Copy link
Member

billzhonggz commented Jul 9, 2018 via email

@jyhi
Copy link
Member Author

jyhi commented Jul 9, 2018

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
Copy link
Member

ecwu commented Jul 9, 2018

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

@billzhonggz
Copy link
Member

billzhonggz commented Jul 9, 2018 via email

@jyhi
Copy link
Member Author

jyhi commented Jul 9, 2018

@billzhonggz Alright then.

@ecwu
Copy link
Member

ecwu commented Jul 9, 2018

@billzhonggz Can you explain the offset parameter?

@billzhonggz
Copy link
Member

billzhonggz commented Jul 9, 2018 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants