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.

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

velocity scale, azimut and elevation bugs patched #2

Merged
merged 1 commit into from
Apr 29, 2016

Conversation

sergiopoppi
Copy link
Member

Velocity scale must have opposite sign of the frequency scale, by convention

Bug solved: instead of azimut and elevation there was the right ascension and the declination

@flyingfrog81
Copy link
Member

Sergio grazie dei cambiamenti!

  1. Velocita ok cambiare il segno
  2. azel ti spiego il ragionamento che avevamo fatto, poi sei tu che sai come funziona class e quindi mi fido del tuo giudizio:
    per la modifica che hai fatto AZ ed EL sono le coordinate puntate, con offset precessione e correzioni varie. Io avevo immaginato invece di dover mettere qualcosa relativo alla sorgente, immaginandomi ad esempio di dover integrare scan diversi nel tempo o roba simile, per cui avevo utilizzato RA e DEC nominali della sorgente puntata.
    Se mi dai conferma che la tua interpretazione e' giusta mergio tutto.

@flyingfrog81
Copy link
Member

flyingfrog81 commented Apr 28, 2016

Ci sono anche i parametri:

obs.head.gen.typec = code.coord.equ
obs.head.pos.lam = self.ra
obs.head.pos.bet = self.dec

Oltre a questo che non ritrovo nella documentazione ma che forse andrebbe cambiato se passiamo ad un frame orizzontale nell'header principale:

obs.head.gen.typec = code.coord.equ

e dalla documentazione vedo anche che ci potrebbe essere questo che non capisco bene cosa possa significare:

obs.head.pos.system = ??? 

@flyingfrog81 flyingfrog81 merged commit 301b421 into discos:master Apr 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants