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.

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

Add L1 data injection RPC #27

Closed
wants to merge 42 commits into from
Closed

Add L1 data injection RPC #27

wants to merge 42 commits into from

Conversation

dstndstn
Copy link
Collaborator

  • adds inject_data RPC to server and python client
  • add a heavy-weight RPC config option
  • factor out replacing network interface names by IP addresses
  • adds an injector transform to the rf_pipelines chain

dstndstn and others added 30 commits October 18, 2018 12:52
@dstndstn
add injector transforms to rfi chains; pass to RPC servers
e0a77b2
@dstndstn
add initial inject_data RPC call
b67c279
@dstndstn
add initial implementation of inject_data to python rpc_client
3d748cc
@dstndstn
move injector initialization to make_rpc_servers, before spawning ded…
781514d 
…ispersion threads
@dstndstn
inject data
4ff69b8
@dstndstn
injected data: use offset in samples, not fpga counts
8c3575a
@dstndstn
Merge branch 'kms_rfimask' into inj+rfi
0c36f52
@dstndstn
test-rfi-mask: inject some data too
1da4d86
@dstndstn
try using the assembled_chunk file writer; correct scale/offset order
b25ce2a
@dstndstn
test-rfi-mask: end streams
db60cde
@dstndstn
Merge branch 'inj+rfi' of https://github.com/kmsmith137/ch_frb_l1 int…
da3a0f4 
…o inj+rfi
@dstndstn
rpc_client: add inject_single_pulse for simpulse pulses
6c0b73c
@dstndstn
Merge branch 'inj+rfi' of github.com:kmsmith137/ch_frb_l1 into inj+rfi
2997c59
@dstndstn
test-rfi: make bigger RFI noise
2d6c42e
@dstndstn
Merge branch 'inj+rfi' of https://github.com/kmsmith137/ch_frb_l1 int…
f0f6773 
…o inj+rfi
@dstndstn
inject a pulse; make plots of intensity data too
0e4e9e2
@dstndstn
add "heavy" argument to L1 RPC server: are we doing heavy-weight RPC …
485d0da 
…calls? & allow empty injectors array.
@dstndstn
fix L1Rpc constructors
275ebba
@dstndstn
factor out creating & using the network interface name map
41fc7ba
@dstndstn
add heavy-weight RPCs to ch-frb-l1
2f0fc42
@dstndstn
revert build flags for simulate_l0.so
047c809
@dstndstn
make heavy_rpc_address config options actually optional
fe07023
@dstndstn
factor out _handle_inject_data function
4b0da37
@dstndstn
testing: add hand-crafted inject_data msgpack implementation
cdf49ba
@dstndstn
try adding a subclass of std::vector<T> that is msgpacked in binary form
7aabcc1
@dstndstn
oops, python msgpack packing was totally wrong
957546f
@dstndstn
add inject_data_request::swap() method to avoid vector copies
28ce57d
@dstndstn
use ch_frb_io/msgpack_binary_vector.hpp; remove hand-coded msgpack va…
b5c6f22 
…riants
@dstndstn
move inject_data_binmsg into rpc.hpp
15d1e20
@dstndstn
rename inject_data_binmsg to inject_data_request
5f5a758
@dstndstn dstndstn mentioned this pull request Nov 18, 2018
Open
@dstndstn dstndstn mentioned this pull request Feb 25, 2019
Open
@dstndstn
Copy link
Collaborator Author

Replaced by #32

@dstndstn dstndstn closed this Sep 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@dstndstn