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

3 ETCS missing #103

Open
reisi007 opened this issue Nov 4, 2015 · 3 comments
Open

3 ETCS missing #103

reisi007 opened this issue Nov 4, 2015 · 3 comments

Comments

@reisi007
Copy link

reisi007 commented Nov 4, 2015

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

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

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

rnbwdsh commented Oct 21, 2019

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

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

No branches or pull requests

3 participants