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

Remove error messages for unknown rooms in the scraper #15

Closed
2 tasks done
blu3r4y opened this issue Oct 10, 2020 · 0 comments
Closed
2 tasks done

Remove error messages for unknown rooms in the scraper #15

blu3r4y opened this issue Oct 10, 2020 · 0 comments
Labels
good first issue Good for newcomers scraper Issues concerning the web scraper component

Comments

@blu3r4y
Copy link
Owner

blu3r4y commented Oct 10, 2020

  • Do not raise errors on unknown rooms, instead, just add a note to the final log message, e.g.
scraped 42 room bookings for course '123456' (of which 12 unknown rooms were ignored: ['tba', 'unknown'])
  • Add a statistic counter on numIgnoredBookings which counts all bookings with unknown room names (non-unique)

Just as a note, here is a list of commonly unknown rooms:

-
n.n.
digital
divers
extern
hotel
kein
kusss
live
mail
moodle
offen
online
open
stream
tba
tbd
teams
todo
unbekannt
unknown
video
virtual
virtuell
zoom
@blu3r4y blu3r4y added scraper Issues concerning the web scraper component good first issue Good for newcomers labels Oct 10, 2020
@blu3r4y blu3r4y closed this as completed Nov 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Good for newcomers scraper Issues concerning the web scraper component
Projects
None yet
Development

No branches or pull requests

1 participant