cro2003 / rgpvApi

Unofficial API Wrapper for RGPV
https://cro2003.github.io/rgpvApi/
GNU General Public License v3.0
2 stars 6 forks source link

Add more features to csvresults function #10

Closed PavanTeja2005 closed 1 month ago

PavanTeja2005 commented 1 month ago

Hi, I want to generalize the csvresults function to be able to read data from both csv and excel files and as the csv reader function reads rows one by one, I want to make it fast such that it fetches multiple results in parallel. 

I'm currently working on GSSoC Extended. Please assign me the labels.

@cro2003

cro2003 commented 1 month ago

Sure, that's a great idea, before that can you elaborate how will you achieve parallel result fetching?

PavanTeja2005 commented 1 month ago

I'm planning to use threads to make multiple requests and fetch data at once.

cro2003 commented 1 month ago

That Works but remember rgpv can deny your requests so make sure to balance that out.

PavanTeja2005 commented 1 month ago

Okay., thank you. I will make sure to keep that in check. And as the library uses pyteserract, I'm planning to add a method to handle the case when the user's system doesn't contain the tesseract software and prompt them with the steps and details to install the software along with the proper links to download the software based on their system.

PavanTeja2005 commented 1 month ago

can you please provide me with test data

cro2003 commented 1 month ago

Sure you can test with below info Enollment Series: 0176CS221001 - 0176CS221200 Course Id: 24 Semester: 4

PavanTeja2005 commented 1 month ago

Can you please check out my pull request..

cro2003 commented 1 month ago

Sure will do.