Skip to content

Add support for dual-purpose dr_tengine.py #54

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
Open
jaycedowell wants to merge 45 commits into
base: main
Choose a base branch
from
Open

Add support for dual-purpose dr_tengine.py #54

jaycedowell wants to merge 45 commits into from

Conversation

@jaycedowell
Copy link
Member

@jaycedowell jaycedowell commented Dec 9, 2024

This PR adds the updated version of dr_tengine.py that allows for recording of both pre and post-T-engine voltage beam data.

@jaycedowell
Allow StatusLogger to look at multiple file queues.
3c5f082
@jaycedowell
Pull in dr_vbeam.py.
1f5f0bd
@jaycedowell
Add on a couple of extra cores.
8a48c9a
@jaycedowell
Tweak the filename for raw voltage recordings.
e9f8efb
@jaycedowell
Down select to RawRecord and BND.
468b715
@jaycedowell
Extra r.
3c8b2e4
@jaycedowell
Try a combined command processor.
d7153ee
@jaycedowell
Time fix plus better logging.
9d6bdcc
@jaycedowell
Override these as well.
54f0091
@jaycedowell
Fix a few things related to monitoring multiple file queues.
d04fadd
@jaycedowell
Another place that needs an update.
5ab96f9
@jaycedowell
Typo.
cddb4e8
@jaycedowell
Ah, this is needed earlier than I thought.
fd6bc47
@jaycedowell
Some day...
322afd4
@jaycedowell
This might finally be it.
b2aa396
@jaycedowell
Nope, there's a typo.
561f088
@jaycedowell
Type problem here as well.
408bab1
@jaycedowell
Start 'server' at 1.
b4e06c8
@jaycedowell
Work on growing the bandwidth that raw_record can output.
e95cbdd
@jaycedowell
Fix the packet burst control.
9a0c5c5
@jaycedowell
Need to range().
7418ccc
@jaycedowell
Use integer division.
08285e4
@jaycedowell
Use the correct integer division.
6da46ff
@jaycedowell
We're sending too much data per pass.
58b4f1f
@jaycedowell
Switch to RBeam format for writing out the raw voltage beam.
b88e886
@jaycedowell
Doc update for RBeam format and new pipeline structures.
9ecce05
@jaycedowell
Add a note about channel numbers and counts.
ae4ceb3
@jaycedowell
Start with server 1 not 2.
9efce39
@jaycedowell
Shape fix.
9dcf7a9
@jaycedowell
Just 'nchan' again.
38013e1
@jaycedowell jaycedowell mentioned this pull request Dec 9, 2024
Open
@caseyjlaw
Copy link
Contributor

caseyjlaw commented Feb 13, 2025

This has been deployed at OVRO-LWA. I also installed an update to allow the lwa-observing scheduler to use it. I think this PR can be closed.

@caseyjlaw
Copy link
Contributor

caseyjlaw commented Feb 13, 2025

I take it back. I was not able to deploy the code as the dr-vslow services now report ModuleNotFoundError: No module named 'ovro_data_recorder'.

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.

3 participants

@jaycedowell @caseyjlaw