cessna172 with [LDY] in wind axes #79

aqreed · 2016-09-12T19:46:35Z

After indentation modifications, addressing of the remaining FIXMEs and including pqr in the model along with the aerodynamics data.

Closes #67

Test of futures class "AeroCoeff" with interp1d interpolation

No longer needed

The value for alpha = 5º is negative, not positive (see Louis Van Gaal)

Cessna aqreed

Merge branch 'master' of https://github.com/aqreed/PyFME Conflicts: src/pyfme/aircrafts/cessna_172.py

codecov-io · 2016-09-12T19:49:28Z

Current coverage is 87.25% (diff: 83.52%)

Merging #79 into master will decrease coverage by 0.26%

@@             master        #79   diff @@
==========================================
  Files            26         28     +2   
  Lines          1746       1914   +168   
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
+ Hits           1528       1670   +142   
- Misses          218        244    +26   
  Partials          0          0

Powered by Codecov. Last update e8c5163...a91f029

astrojuanlu · 2016-09-12T20:20:04Z

Reemplaza a #77

Por mi parte el código está bien y es consistente con el resto del proyecto. Estaría bien tener algún test, aunque sea de algo trivial, para que por lo menos se inicialice el avión. Al margen de eso, a la espera de lo que diga @AlexS12 sobre meter o no el modelo de motor.

AlexS12 · 2016-09-12T21:40:03Z

src/pyfme/aircrafts/cessna_172.py

+    """
+
+
+def __init__(self):


Tienes que indentar un nivel todo el init es lo único que se te ha pasado.

AlexS12 · 2016-09-12T21:44:16Z

Por mi, parte con esos comentarios que han quedado arriba, está todo guay! El único tema clave es el de la indentación del __init__() que se te ha debido ir.
Si te animas a cambiar los ejemplos, perfecto.
Mergeamos cuando esto esté cambiado y añadimos los cambios del modelo del motor en otro pull request sin presión

aqreed · 2016-09-12T21:48:17Z

voy!

AlexS12 · 2016-10-03T17:04:55Z

src/pyfme/aircrafts/cessna_172.py

@@ -251,8 +252,8 @@ def _calculate_aero_lat_forces_moments_coeffs(self):
        CN_delta_aile_interp = interpolate.RectBivariateSpline(self.delta_aile_data, self.alpha_data, self.CN_delta_aile_data)

        self.CY = CY_beta_interp * self.beta + CY_delta_rud_interp * delta_rud_RAD + (b/(2 * V)) * (CY_p_interp * p + CY_r_interp * r)
-        self.Cl = Cl_beta_interp * self.beta + Cl_delta_aile_interp + Cl_delta_rud_interp * delta_rud_RAD + (b/(2 * V)) * (Cl_p_interp * p + Cl_r_interp * r)
-        self.CN = CN_beta_interp * self.beta + CN_delta_aile_interp(delta_aile, alpha_DEG)[0, 0] + CN_delta_rud_interp * delta_rud_RAD + (b/(2 * V)) * (CN_p_interp * p + CN_r_interp * r)
+        self.Cl = 0.5*Cl_beta_interp * self.beta + Cl_delta_aile_interp + 0.1*Cl_delta_rud_interp * delta_rud_RAD + (b/(2 * V)) * (Cl_p_interp * p + Cl_r_interp * r)


Entiendo lo que me comentaste el otro día Andrés, pero ¿estos números se eligen con algún significado físico o variándolos hasta que cuadre otra cosa (algún amortiguamiento o algo así)?

Sentido fisico no tienen, son adimensionales. Se tocan hasta que los valores de pqr a deflexiones maximas no excedan 1-1.5 rad/s, y los amortiguamientos sean razonables (un step o doublet esté prácticamente amortiguado antes de los 10s, por ejemplo). Se hace basándose en la experiencia, viendo horas de videos de la C172 en youtube hasta obtener cosas con sentido.

Los métodos usados son antiguos, si tuviéramos túnel de viento saldría todo a la primera!

aqreed · 2016-10-04T07:41:08Z

Bueno doy por buena la Cessna 172:

ejemplos.py adaptados
Vuelo recto y nivelado
Ascensos y descensos
Doblete timón de profundidad
Doblete con el timón de dirección
Doblete con los alerones
Virajes en horizontal: para turn_rates entre 0.005 - 0.26 rad/s hace cosas con sentido
Virajes en ascenso: funciona en combinaciones adecuadas de gamma0 y turn_rate (por ejemplo turn_rate = 0.25 y gama0 = 0.05)

aqreed · 2016-10-06T03:30:22Z

ya hay test

astrojuanlu · 2016-10-16T11:43:41Z

¡Hora de desbloquear esto! @AlexS12 ¿Necesitas que le dé a esto una revisión de estilo o alguna cosa más? @aqreed Si no me equivoco acabaste tuneando los coeficientes con respecto a lo que salía en DATCOM, ¿es así?

aqreed · 2016-10-16T16:59:03Z

si los resultados siempre hay que ajustarlos. Mira a ver qué cosas se pueden mejorar y me dices

AlexS12 · 2016-10-17T19:37:58Z

Tienes un pull req en aqreed#5

Por mi parte el avión se puede quedar congelado. Voy a echar un vistazo a los ejemplos y hacemos hacer el merge.

Cessna aqreed

AlexS12 · 2016-11-04T17:32:41Z

Bueno, en primer lugar mis disculpas, por haber taradado tanto en resolver esto.

La verdad, es que el nuevo avión ¡ha quedado muy bien!

Hay algunos puntos que tendremos que ir mejorando poco a poco, pero vamos a hacerlo a través de nuevos issues y pull requests para no dejar esto bloqueado.

Así, enhorabuena Andrés!!! 🎉

aqreed and others added 20 commits August 23, 2016 18:35

Add files via upload

842ceec

Merge pull request #1 from aqreed/aqreed-CESSNA172_coeff

34be7e8

Test of futures class "AeroCoeff" with interp1d interpolation

Add files via upload

f93b42e

Add files via upload

7ac0c18

Add files via upload

651ea05

cessna 172 added and object aircraft modified with pqr

69bdd2c

Merge branch 'master' into master

1b15bb8

Fix indentation in aircraft.py

235798c

Fix indentation in cessna_172.py

230920c

Deleted self._long_inputs and self._lat_inputs

acb47f3

No longer needed

Merge remote-tracking branch 'upstream-aqreed/master' into cessna-aqreed

84dc0c8

Corrected self.CM_data

e3c5bf3

The value for alpha = 5º is negative, not positive (see Louis Van Gaal)

minor changes for cessna172

0974d82

Merge pull request #2 from AlexS12/cessna-aqreed

4f77a75

Cessna aqreed

borrada por tocar las narices

a0f1c00

modified coefficient description

d83e901

modified description

1747679

Merge branch 'master' of https://github.com/aqreed/PyFME Conflicts: src/pyfme/aircrafts/cessna_172.py

fixed indentation

460fd80

FIXME on LDY wind2body addressed

464a762

delta_t added to self.controls

d7ff060

AlexS12 reviewed Sep 12, 2016
View reviewed changes

aqreed added 3 commits September 12, 2016 23:49

corrected _init_

be7988b

indentation, again...

df10cb5

indantantion...again2

039d103

aqreed added 5 commits September 28, 2016 22:15

data presentation changes and doublet offset modified

a1d59bd

aileron doublet OK

3e6f264

aileron doublet T=1

10cdc78

modified the dynamics of CM, Cl and CN

6317769

rudder doublet OK

2d3a454

AlexS12 reviewed Oct 3, 2016

View reviewed changes

modified Cl and CN for C172

a52e56a

aqreed added 4 commits October 5, 2016 02:21

modified turn examples

5ced9a6

undone changes to input generator

a0b534d

modified C172 dim feet to meter

2232eda

added test for the C172

99cc856

AlexS12 added 3 commits October 17, 2016 20:56

Minor style change in coeff forumlas

399b730

Some more style changes

82b47be

improved readability

d89bde6

aqreed added 8 commits October 18, 2016 02:52

Merge pull request #5 from AlexS12/cessna-aqreed

8d99757

Cessna aqreed

changes to C172, coeff w/o interp

d77f9d4

C172 coeffs interp modification

9216e2c

modified aileron offset

13d2189

merge

ebad609

C172 coeffs...again

5766258

modified elevator offset

6811b8a

Merge remote-tracking branch 'upstream/master'

a91f029

AlexS12 merged commit effcdfc into AeroPython:master Nov 4, 2016

astrojuanlu mentioned this pull request Nov 8, 2016

Update notebook examples #95

Open

3 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

cessna172 with [LDY] in wind axes #79

cessna172 with [LDY] in wind axes #79

aqreed commented Sep 12, 2016 •

edited by astrojuanlu

codecov-io commented Sep 12, 2016 •

edited

astrojuanlu commented Sep 12, 2016

AlexS12 Sep 12, 2016

AlexS12 commented Sep 12, 2016 •

edited

aqreed commented Sep 12, 2016

AlexS12 Oct 3, 2016

aqreed Oct 4, 2016

aqreed commented Oct 4, 2016 •

edited

aqreed commented Oct 6, 2016

astrojuanlu commented Oct 16, 2016

aqreed commented Oct 16, 2016

AlexS12 commented Oct 17, 2016

AlexS12 commented Nov 4, 2016 •

edited

cessna172 with [LDY] in wind axes #79

cessna172 with [LDY] in wind axes #79

Conversation

aqreed commented Sep 12, 2016 • edited by astrojuanlu

codecov-io commented Sep 12, 2016 • edited

Current coverage is 87.25% (diff: 83.52%)

astrojuanlu commented Sep 12, 2016

AlexS12 Sep 12, 2016

Choose a reason for hiding this comment

AlexS12 commented Sep 12, 2016 • edited

aqreed commented Sep 12, 2016

AlexS12 Oct 3, 2016

Choose a reason for hiding this comment

aqreed Oct 4, 2016

Choose a reason for hiding this comment

aqreed commented Oct 4, 2016 • edited

aqreed commented Oct 6, 2016

astrojuanlu commented Oct 16, 2016

aqreed commented Oct 16, 2016

AlexS12 commented Oct 17, 2016

AlexS12 commented Nov 4, 2016 • edited

aqreed commented Sep 12, 2016 •

edited by astrojuanlu

codecov-io commented Sep 12, 2016 •

edited

AlexS12 commented Sep 12, 2016 •

edited

aqreed commented Oct 4, 2016 •

edited

AlexS12 commented Nov 4, 2016 •

edited