search

calpoly-csai / cal-poly-knowledge-graph

3 stars 9 forks source link

Feature/course scraper #10

Closed IdoPesok closed 2 years ago

IdoPesok commented 2 years ago

Created a new Course Scraper class that fetches the required course data for all CP courses; from the Cal Poly website.

NOTE In the original model the unit data type was an integer. I have changed it to a string since many courses have a range of units, ie. 1-4 units or 1-9 units.

Waidhoferj commented 2 years ago

Hi @IdoPesok, it looks like there is already a course scraper in the queue #9 . I'm going to give them precedence since they got to it first. There are important aspects of your code, like the unit ranges that would be a great addition. Would you like to split these out into a separate PR when #9 is merged? If that's okay with you, I'll close this PR.

Waidhoferj commented 2 years ago

Closed since #9 has been merged