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.

Sign up for GitHub

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

add REVE1, REVE2, IDEA and MKK modules #85

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

add REVE1, REVE2, IDEA and MKK modules #85

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
3ade063
update parse to take new ics css classes into account
marcoroth Jul 1, 2021
144e61f
add `REVE1`, `REVE2`, `IDEA` and `MKK` modules
marcoroth Jul 1, 2021
6a880dc
update data
marcoroth Jul 1, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
24 changes: 24 additions & 0 deletions src/data/modules_ai.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,27 +1,41 @@
{
"ABIZ": "Kernmodul",
"ACBA": "Erweiterungsmodul",
"ADML": "Kernmodul",
"ADS": "Kernmodul",
"AEDCIT": "Zusatzmodul",
"AEDCTO": "Zusatzmodul",
"AEMBS": "Erweiterungsmodul",
"ANGCPH": "Zusatzmodul",
"ANLIS": "Kernmodul",
"AROB": "Erweiterungsmodul",
"ASACPH": "Zusatzmodul",
"ASTAT": "Kernmodul",
"BCHAIN": "Erweiterungsmodul",
"CAEPRO": "Zusatzmodul",
"CG": "Erweiterungsmodul",
"CHMA": "Erweiterungsmodul",
"CRS": "Erweiterungsmodul",
"CSA": "Erweiterungsmodul",
"CYBER": "Erweiterungsmodul",
"DBS": "Kernmodul",
"DIBA": "Kernmodul",
"DIMA": "Erweiterungsmodul",
"DMATH": "Kernmodul",
"DVIZ": "Kernmodul",
"ECOM": "Erweiterungsmodul",
"ENGDBU": "Zusatzmodul",
"ENGFTD": "Zusatzmodul",
"ENGPRA": "Zusatzmodul",
"ENGSDC": "Zusatzmodul",
"ENGTCS": "Zusatzmodul",
"EPRE": "Kernmodul",
"GAME": "Kernmodul",
"GAMEDEV": "Erweiterungsmodul",
"HFD": "Erweiterungsmodul",
"IAVR": "Erweiterungsmodul",
"IOPT": "Kernmodul",
"IOT": "Erweiterungsmodul",
"ISF": "Erweiterungsmodul",
"ISMA": "Erweiterungsmodul",
"ITRE": "Erweiterungsmodul",
Expand All @@ -31,16 +45,26 @@
"MLPW": "Kernmodul",
"MLPW01": "Kernmodul",
"MLPW02": "Kernmodul",
"MOBILSEC": "Erweiterungsmodul",
"MOD": "Erweiterungsmodul",
"MWI1": "Erweiterungsmodul",
"NETW1": "Erweiterungsmodul",
"NETW2": "Erweiterungsmodul",
"NETW3": "Erweiterungsmodul",
"OOP": "Erweiterungsmodul",
"PHKI": "Kernmodul",
"PMB": "Projektmodul",
"PRIVACY1": "Erweiterungsmodul",
"PROG": "Kernmodul",
"PROTCO": "Zusatzmodul",
"REUF": "Erweiterungsmodul",
"RL": "Kernmodul",
"SIOT": "Erweiterungsmodul",
"SQAS": "Erweiterungsmodul",
"SWT": "Erweiterungsmodul",
"SYSSEC": "Erweiterungsmodul",
"UKOM": "Erweiterungsmodul",
"USAB": "Erweiterungsmodul",
"VWL": "Erweiterungsmodul",
"WEBT": "Erweiterungsmodul"
}
18 changes: 16 additions & 2 deletions src/data/modules_i.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
"ANLIS": "Kernmodul",
"ANRECHINDIVID": "Erweiterungsmodul",
"APPE": "Majormodul",
"APPRE": "Erweiterungsmodul",
"AROB": "Erweiterungsmodul",
"ASACPH": "Zusatzmodul",
"ASTAT": "Erweiterungsmodul",
Expand Down Expand Up @@ -47,7 +48,9 @@
"CPLAB": "Erweiterungsmodul",
"CRS": "Erweiterungsmodul",
"CSA": "Erweiterungsmodul",
"CT": "Majormodul",
"CYBER": "Erweiterungsmodul",
"CYBER2": "Erweiterungsmodul",
"DASB": "Majormodul",
"DAVI": "Majormodul",
"DAWA": "Majormodul",
Expand All @@ -63,6 +66,7 @@
"DL4G": "Majormodul",
"DMATH": "Kernmodul",
"DSO": "Erweiterungsmodul",
"DTHINK": "Erweiterungsmodul",
"EAEK": "Erweiterungsmodul",
"EBUSI": "Zusatzmodul",
"EBV": "Erweiterungsmodul",
Expand Down Expand Up @@ -95,10 +99,12 @@
"HMATH": "Erweiterungsmodul",
"I40_BAS": "Zusatzmodul",
"IAVR": "Majormodul",
"IDEA": "Zusatzmodul",
"IMATH": "Erweiterungsmodul",
"IMLAB": "Erweiterungsmodul",
"INFKOL": "Erweiterungsmodul",
"INFSEC+MAN": "Erweiterungsmodul",
"INLP": "Majormodul",
"INTRO": "Erweiterungsmodul",
"IOS": "Erweiterungsmodul",
"IOTHACK": "Erweiterungsmodul",
Expand Down Expand Up @@ -135,7 +141,9 @@
"ME+TE": "Zusatzmodul",
"MEDTECH": "Erweiterungsmodul",
"MICRO": "Erweiterungsmodul",
"MKK": "Zusatzmodul",
"ML": "Majormodul",
"MOBILSEC": "Erweiterungsmodul",
"MOBLAB": "Majormodul",
"MOBPRO": "Erweiterungsmodul",
"MOBSYS": "Erweiterungsmodul",
Expand All @@ -144,9 +152,12 @@
"MTOP": "Erweiterungsmodul",
"NA": "Zusatzmodul",
"NETW1": "Erweiterungsmodul",
"NETW2": "Erweiterungsmodul",
"NETW3": "Erweiterungsmodul",
"OEK": "Zusatzmodul",
"OEK_PWG": "Zusatzmodul",
"OOP": "Kernmodul",
"OSSEC": "Erweiterungsmodul",
"PAWI": "Projektmodul",
"PCP": "Majormodul",
"PEAK": "Zusatzmodul",
Expand All @@ -158,13 +169,16 @@
"PRAXIS_3/6": "Projektmodul",
"PRAXIS_VT": "Erweiterungsmodul",
"PRAXIS_VTZ": "Projektmodul",
"PREN1": "Projektmodul",
"PREN1": "Erweiterungsmodul",
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this doesn't look right, @Lextum please check...

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The parser still works fine. PREN1 is now actually listed in the section "Erweiterungsmodule" for whatever reason.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In that case the required minimum credits for "Projektmodule" and "Erweiterungsmodule" have changed. We need to check that to make sure the calculations are still correct.

Are there still students studying right now where PREN is counted as "Projektmodul"? Because then we have the salad! 😅

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a new module called iPREN1 and iPREN2 which are more focused on IT according to email. We still have the problem, that we can't distinguish when the module was taken and which type it had back then.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then we might need an option to manually change the type of a module.

"PREN2": "Projektmodul",
"PRG": "Kernmodul",
"PROTCO": "Zusatzmodul",
"PTA": "Projektmodul",
"RCCR": "Zusatzmodul",
"REUF": "Erweiterungsmodul",
"REVE1": "Erweiterungsmodul",
"REVE2": "Erweiterungsmodul",
"RL": "Erweiterungsmodul",
"ROBLAB": "Majormodul",
"SELL": "Zusatzmodul",
"SIM+MOD": "Erweiterungsmodul",
Expand All @@ -177,7 +191,7 @@
"SUM_SCHOOL": "Zusatzmodul",
"SWAT": "Majormodul",
"SWDE": "Kernmodul",
"SWT": "Erweiterungsmodul",
"SWT": "Majormodul",
"TE+GE": "Zusatzmodul",
"TECW": "Zusatzmodul",
"TML": "Zusatzmodul",
Expand Down
8 changes: 8 additions & 0 deletions src/data/modules_wi.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,11 +1,13 @@
{
"ABIZ": "Erweiterungsmodul",
"ACBA": "Erweiterungsmodul",
"AEDC1": "Erweiterungsmodul",
"AEDCIT": "Zusatzmodul",
"AEDCTO": "Zusatzmodul",
"AIOT": "Erweiterungsmodul",
"ANGCPH": "Zusatzmodul",
"APMAT": "Kernmodul",
"APPRE": "Erweiterungsmodul",
"ASACPH": "Zusatzmodul",
"ASTAT": "Kernmodul",
"Anwender SAP": "Erweiterungsmodul",
Expand All @@ -24,6 +26,7 @@
"CRS": "Erweiterungsmodul",
"CSA": "Erweiterungsmodul",
"CYBER": "Erweiterungsmodul",
"CYBER2": "Erweiterungsmodul",
"DAMA": "Kernmodul",
"DASB": "Erweiterungsmodul",
"DAVI": "Erweiterungsmodul",
Expand All @@ -33,10 +36,12 @@
"DIBA": "Kernmodul",
"DIBM": "Erweiterungsmodul",
"DICO": "Erweiterungsmodul",
"DIGCRE": "Erweiterungsmodul",
"DILA": "Erweiterungsmodul",
"DIMA": "Erweiterungsmodul",
"DIST": "Erweiterungsmodul",
"DITI": "Erweiterungsmodul",
"DTHINK": "Erweiterungsmodul",
"EAEK": "Kernmodul",
"EALA": "Erweiterungsmodul",
"EAUS": "Kernmodul",
Expand Down Expand Up @@ -81,6 +86,7 @@
"ITRE": "Erweiterungsmodul",
"KBDS": "Erweiterungsmodul",
"KOMA": "Erweiterungsmodul",
"KRKO": "Erweiterungsmodul",
"KRYPTO": "Erweiterungsmodul",
"KUFA": "Erweiterungsmodul",
"MAAC": "Kernmodul",
Expand All @@ -102,6 +108,7 @@
"NPE1": "Projektmodul",
"NPE2": "Projektmodul",
"OOP": "Kernmodul",
"OSSEC": "Zusatzmodul",
"PEFU": "Erweiterungsmodul",
"PMB": "Projektmodul",
"PMRE": "Kernmodul",
Expand All @@ -117,6 +124,7 @@
"STO": "Zusatzmodul",
"SUM_SCHOOL": "Zusatzmodul",
"SWDE": "Kernmodul",
"SWT": "Erweiterungsmodul",
"TUNE": "Zusatzmodul",
"UCDE": "Erweiterungsmodul",
"UEDE": "Erweiterungsmodul",
Expand Down
6 changes: 5 additions & 1 deletion tools/parse_module_type.py
100644 → 100755
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -221,7 +221,7 @@ def parseWebsite(autoDownload=True):

# ics only has "Modulbeschreibungen" --> no loop required for sections
modules = sections[0].find_class(
'columns col-collapse small-12 print-12 download-text')
'columns medium-12 maincontent mobile-column print-full-width')
for module in modules:
module_name = str(etree.tostring(module))
module_id = module_name.rsplit('(',1)[1].rsplit(')')[0].replace('&', '&')
Expand All @@ -240,6 +240,8 @@ def parseWebsite(autoDownload=True):
modules_with_type['WEBLAB'] = majormodul
modules_with_type['KOHEB'] = zusatzmodul
modules_with_type['MAREP'] = repetitoriummodul
modules_with_type['REVE1'] = erweiterungsmodul
modules_with_type['REVE2'] = erweiterungsmodul

# Other modules (that don't appear on the website)
# Sometimes they just rename modules but the names don't change on 'Meine Anmeldungen'
Expand Down Expand Up @@ -315,6 +317,8 @@ def parseWebsite(autoDownload=True):
# ISA modules
modules_with_type['RCCR'] = zusatzmodul # Relax, Concentrate & Create
modules_with_type['NA'] = zusatzmodul # Blockwoche Nachhaltigkeit
modules_with_type['IDEA'] = zusatzmodul
modules_with_type['MKK'] = zusatzmodul
modules_with_type['ENICS1'] = zusatzmodul
modules_with_type['TML'] = zusatzmodul
# Blockwoche Bildnerisches Gestalten
Expand Down