Skip to content

Python homework-0#1

Open
StudentMAGe wants to merge 1 commit intomasterfrom
homework-0
Open

Python homework-0#1
StudentMAGe wants to merge 1 commit intomasterfrom
homework-0

Conversation

@StudentMAGe
Copy link
Copy Markdown
Owner

No description provided.

Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
for X in range(1, s.nrows):
learner = s.row_values(X)
print(learner)
average=sum(learner)
Copy link
Copy Markdown
Owner Author

@StudentMAGe StudentMAGe Sep 17, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"learner" contains text data at position [1], so the sum should be impossible. Another name for the function is needed.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We've redefined sum at line 17 so I'm not sure, that there will be some problems here. Have you tried to run the code?

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Well, the code works, but I thought, that's not what we are talking about here...

Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
a = xlrd.open_workbook("БИОЛОГИ_ таблица успеваемости осень 2018.xlsx")
debug = 1

for s in a.sheets():
Copy link
Copy Markdown
Owner Author

@StudentMAGe StudentMAGe Sep 17, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what is a.sheets()? I just don't get it..

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can try to get it from xlrd package documentation / API reference.

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

Comment thread Untitled.py
Comment thread Untitled.py
Copy link
Copy Markdown
Collaborator

@agbragin agbragin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please, answer the comments.

Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
for X in range(1, s.nrows):
learner = s.row_values(X)
print(learner)
average=sum(learner)
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We've redefined sum at line 17 so I'm not sure, that there will be some problems here. Have you tried to run the code?

Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
a = xlrd.open_workbook("БИОЛОГИ_ таблица успеваемости осень 2018.xlsx")
debug = 1

for s in a.sheets():
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can try to get it from xlrd package documentation / API reference.

Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Comment thread Untitled.py
Copy link
Copy Markdown
Collaborator

@agbragin agbragin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Task accepted. Nice work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants