-
Notifications
You must be signed in to change notification settings - Fork 29
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
reading a file when required. discussion needed. #39
Comments
That's why file read code is outside the url call. |
Typically, the server is run once and then it keeps on running. It is only restarted if something goes wrong or if code needs to be updated in the server. |
but when we will have more question. will we still open all the files? am i getting it right ? |
Yes. In future we will need multiple expected outputs. Maybe at that time
we will simply read all of them before hand and put them in a dictionary
where's the key is the question number and use that.
Or you could have a database and let that software manage when to read and
when to keep in memory.
We will be doing that in a later project. In this project we will only be
introducing multi use and multi question system.
…On Sat 19 Jan, 2019, 16:21 Shivank98 ***@***.*** wrote:
but when we will have more question. will we still open all the files?
for now, only one file is required so we file read code is outside URL
code. but in the future, we will not need all expected_outputs. we will
need expected_outputs only for given question number.
am i getting it right ?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#39 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AHVj0cHE-U-m_U4GnxsKeNjhDt-_hqilks5vEvjJgaJpZM4aJDGu>
.
|
Ohk, That's a lot of space :D
|
Please feel free to open them
…On Sat 19 Jan, 2019, 16:42 Shivank98 ***@***.*** wrote:
Ohk, That's a lot of space :D
-
1. another thing which I just faced is this, don't know how it got
missed.
expected = expected.strip() UnboundLocalError: local variable
'expected' referenced before assignment
while running server.py.
so we will need to add a global expected in url_upload.
-
1. it's giving 'Wrong answer' even I am submitting the right answer.
looks like we are going to need more issues.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#39 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AHVj0WQ8NOZGa417ChhvaQ-i9x6AHxdIks5vEv2ngaJpZM4aJDGu>
.
|
I will positively do it in night. and i will add global expected myself. |
**I am submitting this regarding **
Description of the ISSUE:
currently, the file is read everytime server is run.
I want the file to be read only when it's required and someone is uploading his file to be checked.
The text was updated successfully, but these errors were encountered: