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

Instructor's name differs in two sections #26

Open
tyrinwu opened this issue Aug 13, 2017 · 4 comments
Open

Instructor's name differs in two sections #26

tyrinwu opened this issue Aug 13, 2017 · 4 comments
Assignees

Comments

@tyrinwu
Copy link

tyrinwu commented Aug 13, 2017

See the image below. The instructor on the top right corner says, Jennifer Ansley. But in the description sections, it suggests Schonberg. One may be correct if they are consistent...

Currently, the project uses the instructor's from the description part.

@williamyeny
Copy link
Owner

Thanks for the heads up. We parse all data from the description, so the instructor may be out of date if it is copied from the previous semester. Perhaps it will be better to separate the description and instructor for the most up-to-date instructor data.

@williamyeny
Copy link
Owner

In general, the "Instructor" section lists all possible instructors rather than the particular instructor that semester.

For example: CS201 description says it has instructors Astrachan, Forbes or Rodger. It does not specify which one for this particular semester.

Therefore, an "instructor" button seems necessary. However, I'm not sure if I can find the endpoint for finding instructor data.

@tyrinwu
Copy link
Author

tyrinwu commented Aug 13, 2017

Yes, even in the meeting information section of CS 201, it gives two names, Jeffrey R Forbes, Owen L Astrachan.

I am just thinking that in the instructor section in the meeting information section, it gives full names rather than just last names, which may make searching for professors' information more easily. In the description section, it just gives last names.

Because of the full name reason, I would vote for using the instructor's name from the meeting information section even though it still does not give who exactly teaches the course.

@williamyeny
Copy link
Owner

williamyeny commented Aug 13, 2017 via email

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

No branches or pull requests

2 participants