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

Add/Update Faculty Information #11

Open
aditeyabaral opened this issue Jun 11, 2021 · 4 comments
Open

Add/Update Faculty Information #11

aditeyabaral opened this issue Jun 11, 2021 · 4 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed priority:high High priority issue

Comments

@aditeyabaral
Copy link
Member

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 aditeyabaral added enhancement New feature or request help wanted Extra attention is needed priority:high High priority issue labels Jun 11, 2021
@aditeyabaral aditeyabaral pinned this issue Jun 11, 2021
@aditeyabaral
Copy link
Member Author

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.

@aditeyabaral aditeyabaral changed the title [IMPROVEMENT] Add Faculty Information Add/Update Faculty Information Jun 30, 2021
@aditeyabaral aditeyabaral added the good first issue Good for newcomers label Jun 30, 2021
@Dominastorm
Copy link

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
Copy link
Member Author

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

@Dominastorm
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed priority:high High priority issue
Projects
None yet
Development

No branches or pull requests

2 participants