Skip to content
This repository has been archived by the owner on Mar 4, 2021. It is now read-only.

add files from protozmq #61

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

add files from protozmq #61

wants to merge 2 commits into from

Conversation

dneise
Copy link
Member

@dneise dneise commented Mar 25, 2019

Work in progress

Starting to add protozmq into this repo, so one gets both packages together.

These files are in principle duplicated in both packages protozmq and protozfits

CoreMessages_pb2.py
L0_pb2.py
R1_DigiCam_pb2.py
R1_LSTCam_pb2.py
R1_NectarCam_pb2.py
R1_pb2.py

but it turns out, they are not exactly identical. So I guess one of them is outdated.

@cta-observatory cta-observatory deleted a comment from pep8speaks Mar 25, 2019
@pep8speaks
Copy link

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

Line 41:14: E221 multiple spaces before operator
Line 60:34: E712 comparison to True should be 'if cond is True:' or 'if cond:'
Line 73:80: E501 line too long (91 > 79 characters)
Line 99:1: E305 expected 2 blank lines after class or function definition, found 1
Line 137:1: E305 expected 2 blank lines after class or function definition, found 1
Line 151:1: W391 blank line at end of file

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants