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

Issues when making GET requests to the "/api/v1/courses/search/{university}" endpoint #35

Open
matthew-boisvert opened this issue Oct 26, 2020 · 5 comments

Comments

@matthew-boisvert
Copy link

matthew-boisvert commented Oct 26, 2020

For UCSC, discussion sections are not included in the course information provided. Also, when using CalPoly as the selected university, labs and classes both show up as "Class"

@matthew-boisvert matthew-boisvert changed the title When making GET requests to the "/api/v1/courses/search/{university}" endpoint there seems to be a few issues. For UCSC, discussion sections are not included in the course information provided. Also, when using CalPoly as the selected university, labs and classes both show up as "Class" Issues when making GET requests to the "/api/v1/courses/search/{university}" endpoint Oct 26, 2020
@tisuela
Copy link
Owner

tisuela commented Oct 26, 2020

UCSC discussion sessions

There were some limitations with using a student-made UCSC API. However, there is a work around, and I can probably patch it in a week.

CalPoly labs+classes showing up as class

This is a naming convention error, mostly due to my thought process. All meetings of a course, whether they are labs, classes, lectures, or seminars, will show up as a Class. In a future implementation, I'll be changing it to Section, since that seems to be a more appropriate term.

For you @matthew-boisvert

Frankly, I didn't think anyone was using my tool LOL, but I am still working with a friend to implement other endpoints, albeit slowly. If there's anything you'd like me to prioritize, since it seems you are using it, lemme know! I'd be happy to implement or improve endpoints if it helps on a project you are working on

@matthew-boisvert
Copy link
Author

Thanks for responding so quickly! I actually was able to get section information using SlugScheduler directly for UCSC course data (there is a separate endpoint that captures this data) for my own related app.

I was actually using your API to try and get CalPoly and UCI course info for a lab project I'm working on at UCSC. If it would be possible to classify the labs at CalPoly as sections instead of classes that would be very helpful. Also, I was wondering if it's possible to access the days of the week that a class occurs using your API. Thanks!

@tisuela
Copy link
Owner

tisuela commented Oct 26, 2020

Gotchu! I started implementing a days-of-the-week attribute of classses/section, using datetime objects. I'll be working on it this week, so hopefully by Friday, you can hear back from me with changes for classification of labs, as well as class occurrences throughout the week.

As a sidenote, there is another UCI Web API I'm working on with a team of other students (which is why CourseCake has stagnated). Unfortunately, all documentation went offline due to migration, but I can update you on that as well!

@matthew-boisvert
Copy link
Author

Awesome that's really helpful! Feel free to take your time on those if it takes longer than expected to implement. And once again thank you so much, it has definitely been challenging to find accessible course data for California universities

@matthew-boisvert
Copy link
Author

I also noticed that some of the "time" fields associated with classes from the endpoint /courses/search/uci seem to be full intervals (like "12:00-12:50p") whereas others are just an individual time (like "5:50p"). I was wondering if it would be possible to get the full intervals for the classes

Repository owner deleted a comment from jackhoang1 Feb 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants