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

Fix bracket position issue #62 #63

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

NewJerseyStyle
Copy link

@NewJerseyStyle NewJerseyStyle commented Apr 28, 2024

Fixing issue #62
Change MiscDemo/connect/python/connector.py Line 91
print("{}\t{}\t{}").format(siteid, citycode, pv)
to
print("{}\t{}\t{}".format(siteid, citycode, pv))

The code was from Beijing Dingshi Zongheng Technology Co., Ltd.
It was trying to print some information in a given format. However this intention was not present correctly according to Python syntax. To make it better represent the intention and allow the Python interpreter to execute it correctly, the closing bracket of function call print has been moved to the end of the line.

It is not a original creation of mine, somehow it is done through me.

Line 91
`    print("{}\t{}\t{}").format(siteid, citycode, pv)`
to
`    print("{}\t{}\t{}".format(siteid, citycode, pv))`

Signed-off-by: Stupid <[email protected]>
@CLAassistant
Copy link

CLAassistant commented Apr 28, 2024

CLA assistant check
All committers have signed the CLA.

@NewJerseyStyle
Copy link
Author

I am not sure if my contributions in this pull request is my original creation.
But it is an issue that should be fixed.
Thanks

@DanRoscigno
Copy link
Contributor

Thanks @NewJerseyStyle , can you please sign the CLA?

@NewJerseyStyle
Copy link
Author

@DanRoscigno Thanks. I am trying to identify the complete details of the source and of any license or other restriction related to function and bracket in Python syntax. I will agree CLA once the document is ready.

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.

3 participants