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

Update Examples on RTD #1333

Closed
wants to merge 3 commits into from
Closed

Conversation

zm711
Copy link
Contributor

@zm711 zm711 commented Oct 13, 2023

@apdavison,

Where are the rtd logs for Neo? I'm trying to update some of the examples so I want to see why they are mostly failing on RTD (only one works currently).
I'm also not super familiar with using Group/ChannelView since I mostly use Neo from the rawio layer, but I'll try to add that into this example soon.

@pep8speaks
Copy link

pep8speaks commented Oct 13, 2023

Hello @zm711! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 35:1: W293 blank line contains whitespace
Line 38:40: E231 missing whitespace after ','
Line 62:1: W293 blank line contains whitespace
Line 85:8: E225 missing whitespace around operator
Line 94:34: E231 missing whitespace after ','
Line 94:55: E231 missing whitespace after ','

Comment last updated at 2023-10-13 15:23:38 UTC

@zm711
Copy link
Contributor Author

zm711 commented Oct 13, 2023

I'll fix pep 8 stuff at the end.

@apdavison
Copy link
Member

@zm711 The rtd logs are here: https://readthedocs.org/projects/neo/builds/

@apdavison apdavison added this to the future milestone Jan 26, 2024
@zm711
Copy link
Contributor Author

zm711 commented Mar 7, 2024

Moved this to #1429.

@zm711 zm711 closed this Mar 7, 2024
@zm711 zm711 deleted the examples-update branch April 6, 2024 20:49
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