search

BradleyCai / ucr-schedule-visualizer

Generates an hour by hour view and shows the locations of all the classes from a raw class list from GROWL.
https://waa.ai/ucrsv
MIT License
5 stars 2 forks source link

does not return back correct information for quarter #60

Closed blamejoel closed 8 years ago

blamejoel commented 9 years ago

Pasted in Winter schedule and Schedule Visualizer generated a schedule with what appeared to be information for the same sections for another quarter.

After further investigation it appears to just not parse all of the course info. For example I see discussions for two of my classes, but the lectures did not transfer over.

emmiegit commented 9 years ago

Like you mentioned, it's likely an issue with the parsing. Could you post the input you provided, or if you don't want it visible online, email it to myself or Bradley? Our emails and public keys are available here.

Also, if it's not too difficult, can you provide a screenshot? Thanks.

blamejoel commented 9 years ago

It might be related to the building where the courses are located. Both of the courses that don't show up are in a UV THE room. I'll email both of you the schedule input to try out. I was going to try and debug it and let you know exactly where it skips the courses, but I should do my hw first =D.

emmiegit commented 9 years ago

Sadly, the school website's list is missing a few, so it might be a building we don't know about. If you replace the building with one that our system does recognize, like UNLH, does it work?

blamejoel commented 9 years ago

Bingo, that's it. If I change the suspected courses with the same building and room as one of my other classes, they all show up.

blamejoel commented 9 years ago

Looks like it's not the building, but the room. It'll take the UV building, but doesn't seem to care for the THE# room format. If I just give it a schedule with the # instead of the THE# that GROWL provides, it works.

emmiegit commented 9 years ago

I didn't know GROWL generated room numbers in that format. Send the schedule input and we'll have a fix out for it soon. Thanks for the bug report.

blamejoel commented 9 years ago

Yep, apparently for the UV Theatre rooms.

BradleyCai commented 8 years ago

This bug is now a part of issue #63 and will therefore be closed.