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

Hiding courses that aren't running #60

Open
qaisjp opened this issue Oct 7, 2018 · 4 comments
Open

Hiding courses that aren't running #60

qaisjp opened this issue Oct 7, 2018 · 4 comments
Labels

Comments

@qaisjp
Copy link
Member

qaisjp commented Oct 7, 2018

Some courses disappear and appear. Courses that aren't running are marked as archived. Archived courses are hidden from the website (even when viewing "all").

Archiving courses manually is a wee bit tedious, so I'd like to derive this information from the course.inf.ed.ac.uk data dump.

The idea is to mark a course as "no longer running" if no such course exists for that acronym (if the course acronym is provided).

What's the problem?

Permalinks use the acronym (Compiling Techniques: https://betterinformatics.com/inf3#ct), and external courses that don't have acronyms defined fall back on a slugified title (Speech Processing: https://betterinformatics.com/inf3#sp-speech-processing).

A fix for this is to provide a made up (but humanly common) course acronym for courses like ILA and CAP to have a nice permalink (/inf1#ila and /inf1#cap).

Speech Processing is a bit more weird since SP is the acronym for Secure Programming. The acronym for this one could be changed to speech (instead of the duplicate sp as it is now). I think sp as an acronym for this course was just made up.

Thoughts?

@qaisjp qaisjp added the question label Oct 7, 2018
@sleepyfoxen
Copy link
Member

The sluggified link is also fine, though. Did you report the acronym overlap to inf?

@qaisjp
Copy link
Member Author

qaisjp commented Oct 8, 2018

Speech Processing is not an informatics course

@sleepyfoxen
Copy link
Member

oh, I see. In that situation it may be possible to use inf3#ppls-sp as a general way to avoid ambiguity

@qaisjp
Copy link
Member Author

qaisjp commented Oct 8, 2018

that works perfectly 🙂

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

No branches or pull requests

2 participants