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.

Sign up for GitHub

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

Jump to bottom

Remove serializer code in #887

Closed

kbruegge opened this issue Dec 12, 2018 · 0 comments

Assignees

Member

kbruegge commented Dec 12, 2018

This came up while looking at /ctapipe/io/tests/test_serializer.py

kbruegge assigned kosack

maxnoe added a commit to maxnoe/ctapipe that referenced this issue


          Remove serializer, fixes cta-observatory#887

88ee457

maxnoe mentioned this issue

Remove serializer, fixes #887 #913

Merged

maxnoe closed this as completed in #913

maxnoe added a commit that referenced this issue


          Remove serializer, fixes #887 (#913)

watsonjj added a commit to watsonjj/ctapipe that referenced this issue


          Merge branch 'master' into factory_v3

7d0f868

* master:
  Major coordinate refactoring, fixes cta-observatory#899, cta-observatory#900, cta-observatory#505, cta-observatory#416 (cta-observatory#896)
  speed up is_compatible; makes event_source factory faster (cta-observatory#927)
  Prefix containers (cta-observatory#833)
  remove optional deps in order to fix tests (cta-observatory#925)
  Charge Resolution Update (cta-observatory#827)
  Remove serializer, fixes cta-observatory#887 (cta-observatory#913)
  deleted summary.html - accidentally committed file (cta-observatory#919)

# Conflicts:
#	ctapipe/tools/extract_charge_resolution.py
#	examples/simple_event_writer.py

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