search

HackerSpace-PESU / pesu-academy-bot

PESU Academy Discord Bot built for PESsants and PESts of PES University
http://bit.ly/pesu-academy-bot
21 stars 9 forks source link

Add/Update Faculty Information #11

Open aditeyabaral opened 3 years ago

aditeyabaral commented 3 years ago

Add a command that lets you retrieve information about faculty. Should allow you to list names of professors and their email addresses based on course as well as branch. For example, pes.faculty cse could list all the professors in CSE and pes.faculty cse daa could list all the CSE professors taking up DAA. Additionally, incorporate campus choice into this as well - pes.cse faculty rr cse daa and so on.

This would obviously mean that this list would have to be hardcoded (as well as updated and maintained every semester since the course names can keep changing). Any suggestions to automate this task are welcome, although I do not think this is possible in any way.

aditeyabaral commented 3 years ago

The feature has been added to the bot however it still lacks data to function. To contribute to this feature, simply add faculty details to data/faculty.csv and open a PR. Once merged the bot will be able to use the newly added details.

Dominastorm commented 2 years ago

I believe this can be automated to some extent. All of the professors' details are present in staff.pes.edu. You can scrape this website to make the .csv file containing all the info. In the teaching column on each of the professors' page is a list of all the courses that they are taking. So, all that would be needed to done is have a list of all the courses and their corresponding course acronyms (design and analysis of algorithms - daa). When the program will read design and analysis of algorithms, it can append daa to the list of the courses that the professor takes.

So, you won't have to hard-code all the faculty details, instead just keep the course names and their acronyms updated (if they do change that is)

aditeyabaral commented 2 years ago

Oh looks like they have updated it. This information wasn't there before. Think you can add the feature?

Dominastorm commented 2 years ago

Pretty busy atm, but as soon as I get time, i can definitely do it.