search

marunjar / anewjkuapp

GNU General Public License v3.0
16 stars 4 forks source link

3 ETCS missing #103

Open reisi007 opened 9 years ago

reisi007 commented 9 years ago

Hi,

I just realized that 3ETCS are missing in your app (and in the excel calculation I made)

The mistake I made was to convert 4,5 ETCS to 4. Could you please check, that you did indexof '/' and not indexof ',' which was causing the mistake in Excel.

Thank you :)

marunjar commented 9 years ago

hi

are you talking from courses (they are parsed from my registrations) or from the results of your exams?

ects/sws should be split by '/' so this should be fine.

what locale do you use, maybe there're some problems?

reisi007 commented 9 years ago

I use a German Android. Although it's not good to mix it up, I only have 9 ETCS "open" this semester. It should be 28. This 9 ETCS are "Begründete Abmeldung"

I have 125 ETCS which seems to lack 0,5 ETCS from 2 UE and VO.... So parsing the ETCS I think that 4,5 is counted as 4

rnbwdsh commented 5 years ago

Works for me, should be handled by Assessment.java#L135 so i guess this can be closed?