-
Notifications
You must be signed in to change notification settings - Fork 1
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
T program no longer writes velocities to .kel files #2
Comments
Hi Dave,
Sorry about any confusion.
Now I’m wondering what happened. I think someone might have asked for frequencies
instead of velocities. It should not be too hard to make the output match the requested
plot type.
Did you want all velocities in the spectrum or only the velocities plotted?
The difference between the output files and the plots has been a little bit of
a documentation headache.
If frequencies, then all data were written+plotted. If velocities, only the
selected velocities were plotted (and written?).
Best regards
Glen
I’ve got lots of meetings, so this may take a day or two.
Glen I Langston, Ph. D.
Galactic Astronomy Program Director
National Science Foundation
304-456-3032
… On May 25, 2022, at 1:45 PM, dave290 ***@***.***> wrote:
This email originated from outside of the National Science Foundation. Do not click links or open attachments unless you recognize the sender and know the content is safe.
Hello! Up until recently, the T program would write velocities (instead of frequencies) to the .kel file, when the -V and -W flags were used. I updated the analyze package a few months ago and just noticed that instead of velocities, frequencies (in Hz) are written instead, even though the labels within the .kel file say 'velocity.' I did a git pull this morning and see the same issue.
For example, you should see it if you run T with the data files that are provided in the complimentary data set (18-11-01T*...18-11-02T*) using the command below:
T -B -L -200 -H +200 -V -W 600 *
and other related commands, like:
T -W 600 *
When the .kel file is written, the data lists frequencies instead of velocities (see example below):
N Velocity Intensity
(m/sec) (Kelvins)
0000 1419981933 -0.4236
Note that T does, in fact, plot the velocity range correctly.
I dove into the code, but it's not obvious to me what to modify.
Could you please take a look? Thank you!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.
|
Hi Glen!
Yes, it’s fine if only velocities within selected range are written to
file. I think that’s the way it was before.
Thank you so much,
Dave
On Wed, May 25, 2022 at 1:00 PM Glen Langston ***@***.***>
wrote:
… Hi Dave,
Sorry about any confusion.
Now I’m wondering what happened. I think someone might have asked for
frequencies
instead of velocities. It should not be too hard to make the output match
the requested
plot type.
Did you want all velocities in the spectrum or only the velocities
plotted?
The difference between the output files and the plots has been a little
bit of
a documentation headache.
If frequencies, then all data were written+plotted. If velocities, only
the
selected velocities were plotted (and written?).
Best regards
Glen
I’ve got lots of meetings, so this may take a day or two.
Glen I Langston, Ph. D.
Galactic Astronomy Program Director
National Science Foundation
304-456-3032
> On May 25, 2022, at 1:45 PM, dave290 ***@***.***> wrote:
>
> This email originated from outside of the National Science Foundation.
Do not click links or open attachments unless you recognize the sender and
know the content is safe.
>
>
>
>
> Hello! Up until recently, the T program would write velocities (instead
of frequencies) to the .kel file, when the -V and -W flags were used. I
updated the analyze package a few months ago and just noticed that instead
of velocities, frequencies (in Hz) are written instead, even though the
labels within the .kel file say 'velocity.' I did a git pull this morning
and see the same issue.
>
> For example, you should see it if you run T with the data files that are
provided in the complimentary data set (18-11-01T*...18-11-02T*) using the
command below:
> T -B -L -200 -H +200 -V -W 600 *
> and other related commands, like:
> T -W 600 *
>
> When the .kel file is written, the data lists frequencies instead of
velocities (see example below):
>
> N Velocity Intensity
>
> (m/sec) (Kelvins)
>
> 0000 1419981933 -0.4236
>
> Note that T does, in fact, plot the velocity range correctly.
> I dove into the code, but it's not obvious to me what to modify.
> Could you please take a look? Thank you!
>
> —
> Reply to this email directly, view it on GitHub, or unsubscribe.
> You are receiving this because you are subscribed to this thread.
>
—
Reply to this email directly, view it on GitHub
<#2 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AP25FZ2NER2I4STMX7GQWGTVLZTCLANCNFSM5W54UR5A>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
...Or, whichever is easier for you, in terms of which velocities are written to file. |
Hey Glen, Just checking in about the velocity-writing issue with T.py Thanks! |
Thank you so much for updating the code with the fix. Works just fine again! |
Hi Dave,
Glad to hear I could help your student.
We’re trying to expand our program to more high schools.
It would be great if you could get us a few pictures of you,
your telescope and some students.
Thanks!
Glen
Glen I Langston, Ph. D.
Galactic Astronomy Program Director
National Science Foundation
304-456-3032
… On Jun 12, 2022, at 1:29 PM, dave290 ***@***.***> wrote:
This email originated from outside of the National Science Foundation. Do not click links or open attachments unless you recognize the sender and know the content is safe.
Thank you so much for updating the code with the fix. Works just fine again!
Best,
Dave
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you commented.
|
Hello! Up until recently, the T program would write velocities (instead of frequencies) to the .kel file, when the -V and -W flags were used. I updated the analyze package a few months ago and just noticed that instead of velocities, frequencies (in Hz) are written instead, even though the labels within the .kel file say 'velocity.' I did a git pull this morning and see the same issue.
For example, you should see it if you run T with the data files that are provided in the complimentary data set (18-11-01T*...18-11-02T*) using the command below:
T -B -L -200 -H +200 -V -W 600 *
and other related commands, like:
T -W 600 *
When the .kel file is written, the data lists frequencies instead of velocities (see example below):
N Velocity Intensity
(m/sec) (Kelvins)
0000 1419981933 -0.4236
Note that T does, in fact, plot the velocity range correctly.
I dove into the code, but it's not obvious to me what to modify.
Could you please take a look? Thank you!
The text was updated successfully, but these errors were encountered: