center-randomize
center-randomize copied to clipboard
moest-np
Download Results as pdf feature was added to the streamlit webapp
For downloading feature I don't think columns like longitude and latitude are needed👍🏻
Also add note specifying what is scode and cscode at the bottom in pdf
For downloading feature I don't think columns like longitude and latitude are needed👍🏻
Also add note specifying what is scode and cscode at the bottom in pdf
Need recommendations on this. Shall I make changes as mentioned above?
Hey @ShudarsanRegmi, I have a few suggestions:
- Allow the user to view the downloaded directly from the UI itself. A simple toast with file URL should do the trick. I'm not sure though.
- Store the downloaded file in a folder named something like
exportsorgenearated-pdfs - Reduce the margins in the header to be bare minimum. Currently, there's too much gap between the elements in the heading section.
- Only display required columns in PDF as suggested by @LuluW8071
- Consider the linting of the code in order to make the code more standard.
@dhirajraut1
Is it okay now? Do you recommend any further changes?
Shall I try moving the logo to the left or it is fine?
Looks good to me.
I have made some changes that allows the user to download the files. I'd be more than happy to collaborate.
- Allow the user to view the downloaded directly from the UI itself. A simple toast with file URL should do the trick. I'm not sure though.
By this do you mean viewing the contents of pdf from streamlit?
By this do you mean viewing the contents of pdf from streamlit?
@LuluW8071 I meant the user to be able to download the file to their desired file location. Non-technical user don't have to see the file in the source directory. They can simply download the file directly. I have created something similar. See the screenshot below.
@dhirajraut1 Is this project even for distribution for general users? This is meant to be run by the exam conducting comittee, isn't it? Why to worry too much about the download path ?
User can now View the downloaded file directly in the browser. The rendered pdf can again be downloaded to any desired location.
Further. Irrelevant columns from both the pdfs were removed and the generated files are stored in generated-pdfs folder.
Margin issue on header section was also fixed.
Thank you for your effort but I see few fundamental issues with this feature -
- There is a manual verification step which cannot be eliminated at this point. Therefore the generated by this app is at best intermediate, does not make sense to have that in print friendly format like pdf
- See center allocation notice published by NEB it contains centers from all districts and crucially it contains registration number distributions for schools. Script must be able to distribute registration numbers before generated PDF would be useful.
Also note this is owned by NEB and not by MoEST directly, therefore MoEST logos would be inappropriate
@sapradhan It may not be ready for final
- There is a manual verification step which cannot be eliminated at this point. Therefore the generated by this app is at best intermediate, does not make sense to have that in print friendly format like pdf
The script is supposed to run again and again until all the students of all schools have not been assigned a center and if the distribution of centers is uneven. If the script gives tne best of its output then It will have to be reviewed by many other members and there will be manual allocations further. For this purpose alone it may be useful, isn't it ? Instead of sharing files like csv or tsv, better pdf can be distributed among the reviewing members.
Instead of sharing files like csv or tsv, better pdf can be distributed among the reviewing members.
CSV and TSV are more suitable for review, edits as they can be easily dealt with spreadsheet applications. This is also for only one district, merging with other districts will also be necessary