Join two loops, which is actually necessary, since we need some information from the start of the loop in the second loop.
Join two loops, which is actually necessary, since we need some
information from the start of the loop in the second loop.

The mail that explains this is (sorry, only german):

---------------------------------
der shapes-Test geht bei mir fehl, und der Grund ist dass in einem
FESystem die _Werte_ nicht von den Basiselementen kopiert werden. Jetzt
habe ich diesen Teil gestern geaendert, so dass das wohl an mir liegt,
aber ich weiss nicht, warum es vorher funktioniert hat.

Also, wie haben ein Q2xQ3 FESystem. Fuer beide muss update_values nicht
in update_each gesetzt werden, weil sich die Funktionswerte ja nicht
aendern. Aber: irgendwann muss es halt mal ins FESystem kopiert werden.
Wo/wann passiert das?

Wenn ich naemlich das erste Mal in FESystem::compute_fill lande (von
FESystem::reinit auf der ersten Zelle), dann habe ich hier (das ist der
alte Code, der neue ist etwas anders, aber im wesentlichen der selbe):

      for (unsigned int base_no=0, comp=0; base_no<n_base_elements(); ++base_no)
        {
          const FiniteElement<dim> &base_fe=base_element(base_no);
          typename FiniteElementBase<dim>::InternalDataBase &
            base_fe_data = fe_data.get_fe_data(base_no);
          const UpdateFlags base_flags(dim_1==dim ?
                                       base_fe_data.current_update_flags() :
                                       base_fe_data.update_flags);
[da FEValues ist hier dim_1==dim]

[... und weiter unten beim kopieren:]
                  if (base_flags & update_values)
                    data.shape_values(system_index, point)=
                      base_data.shape_values(k,point);

Hier enthaelt base_flags kein update_values mehr, so dass die Werte nicht
kopiert werden.

Da das die einzige Stelle in fe_system ist, wo data.shape_values gesetzt
werden, macht mich das ganze ein bisschen stutzig: da kann doch nur was
kopiert werden, wenn die base_flags fuer diese Zelle update_values
enthalten. Also, nachgeschaut:  base_fe_data.update_once hat
update_values, aber base_fe_data.first_cell==false...

Ah, jetzt sehe ich auch was passiert: vorher hatten wir nur eine einzige
Schleife ueber die Basiselement, und in der haben wir die Flags geholt,
das Element aufgefordert sich zu initialisieren, und gleich kopiert.

Jetzt haben wir zwei Schleifen: einmal initialisieren, dann kopieren. Ich
weiss damit auch, wie ich das Problem loesen kann (allerdings erst nach
dem Mittagessen), aber deshalb mal die Frage: ob wir auf dem ersten
Element sitzen, ist eigentlich nicht damit abgehakt, dass sich das
Basiselement initialisiert hat. Trotzdem machen alle Elemente das am Ende
des ersten Aufrufs von fill_fe_*_values. Ist das der richtige Platz?

Und: ganz kurios macht das das FESystem, naemlich irgendwo in der Mitte
der compute_fill-Funktion (die von den fill_fe_*_values Funktionen
aufgerufen wird), und zwar  *im Innern eines Blocks, der in
  if (flags & (update_values | update_gradients))
steht*!! Das will mir eigentlich nicht ganz richtig scheinen...

Ich werde nachher mal einen vorlaeufigen Fix einchecken, damit es
wenigstens wieder geht, aber wuerde trotzdem gerne Eure Meinung dazu
hoeren!
-------------------------------------

...and...

-------------------------------------
Zu ersterer: die update_flags werden auf eine absolut undurchschaubare Art
und Weise gesetzt, geaendert, von irgendwelchen Funktionen zurueckgegeben,
etc. Ich habe bis heute nicht verstanden, welche Funktion ich denn wann
aufrufen muesste, und dass Flags wie first_cell waehrend einer laufenden
(aeusseren) Funktion beeinflussen, was man von current_flags
zurueckbekommt, passt da sehr gut ins Bild.

Ich meine, mal ehrlich: ist es logisch, dass wenn ich current_update_flags
aufrufe, dann fill_fe_values, dann noch mal current_update_flags, dass das
Ergebnis dann ein anderes ist? fill_fe_values ist eigentlich ein typisches
Beispiel einer Funktion, die _Daten_, nicht den _Zustand_ eines Objektes
aendert/aendern sollte! Ich glaube, die Funktion hat bisher eigentlich nur
aus Zufall funktioniert, weil eine lokale Variable oben in einem Block
deklariert und definiert wurde, waehrend sie erst unten benutzt wurde.
Haette man die Deklaration dahin getan, wo die Variable das erste Mal
benutzt wird, dann haette das schon nicht mehr funktioniert.....
--------------------------------


git-svn-id: 31d9d2f6432a47c86a3640814024c107794ea77c@6443 0785d39b-7218-0410-832d-ea1e28bc413d

JaeryunYim/dealii,rrgrove6/dealii,gpitton/dealii,YongYang86/dealii,flow123d/dealii,lpolster/dealii,sriharisundar/dealii,johntfoster/dealii,kalj/dealii,maieneuro/dealii,kalj/dealii,rrgrove6/dealii,flow123d/dealii,sriharisundar/dealii,natashasharma/dealii,Arezou-gh/dealii,lue/dealii,sairajat/dealii,mac-a/dealii,andreamola/dealii,lue/dealii,natashasharma/dealii,adamkosik/dealii,mtezzele/dealii,YongYang86/dealii,sairajat/dealii,lpolster/dealii,gpitton/dealii,EGP-CIG-REU/dealii,lpolster/dealii,natashasharma/dealii,spco/dealii,ESeNonFossiIo/dealii,sriharisundar/dealii,lpolster/dealii,msteigemann/dealii,lpolster/dealii,pesser/dealii,lue/dealii,pesser/dealii,maieneuro/dealii,shakirbsm/dealii,angelrca/dealii,jperryhouts/dealii,shakirbsm/dealii,spco/dealii,kalj/dealii,lpolster/dealii,Arezou-gh/dealii,msteigemann/dealii,flow123d/dealii,sriharisundar/dealii,danshapero/dealii,angelrca/dealii,spco/dealii,adamkosik/dealii,Arezou-gh/dealii,mac-a/dealii,nicolacavallini/dealii,johntfoster/dealii,ibkim11/dealii,EGP-CIG-REU/dealii,shakirbsm/dealii,ibkim11/dealii,naliboff/dealii,nicolacavallini/dealii,ibkim11/dealii,johntfoster/dealii,gpitton/dealii,ESeNonFossiIo/dealii,angelrca/dealii,adamkosik/dealii,andreamola/dealii,ESeNonFossiIo/dealii,sriharisundar/dealii,nicolacavallini/dealii,ESeNonFossiIo/dealii,danshapero/dealii,lue/dealii,flow123d/dealii,andreamola/dealii,gpitton/dealii,adamkosik/dealii,pesser/dealii,msteigemann/dealii,sriharisundar/dealii,msteigemann/dealii,shakirbsm/dealii,mtezzele/dealii,mtezzele/dealii,mtezzele/dealii,naliboff/dealii,ibkim11/dealii,mac-a/dealii,kalj/dealii,JaeryunYim/dealii,spco/dealii,rrgrove6/dealii,Arezou-gh/dealii,YongYang86/dealii,maieneuro/dealii,rrgrove6/dealii,maieneuro/dealii,natashasharma/dealii,shakirbsm/dealii,spco/dealii,kalj/dealii,maieneuro/dealii,adamkosik/dealii,msteigemann/dealii,JaeryunYim/dealii,sairajat/dealii,kalj/dealii,JaeryunYim/dealii,maieneuro/dealii,Arezou-gh/dealii,mac-a/dealii,YongYang86/dealii,mtezzele/dealii,lue/dealii,ESeNonFossiIo/dealii,johntfoster/dealii,EGP-CIG-REU/dealii,jperryhouts/dealii,mac-a/dealii,msteigemann/dealii,nicolacavallini/dealii,mtezzele/dealii,jperryhouts/dealii,gpitton/dealii,naliboff/dealii,angelrca/dealii,natashasharma/dealii,EGP-CIG-REU/dealii,danshapero/dealii,maieneuro/dealii,nicolacavallini/dealii,pesser/dealii,ibkim11/dealii,jperryhouts/dealii,rrgrove6/dealii,andreamola/dealii,naliboff/dealii,danshapero/dealii,andreamola/dealii,ESeNonFossiIo/dealii,YongYang86/dealii,flow123d/dealii,adamkosik/dealii,naliboff/dealii,sairajat/dealii,YongYang86/dealii,Arezou-gh/dealii,nicolacavallini/dealii,andreamola/dealii,adamkosik/dealii,JaeryunYim/dealii,ibkim11/dealii,danshapero/dealii,EGP-CIG-REU/dealii,JaeryunYim/dealii,danshapero/dealii,EGP-CIG-REU/dealii,EGP-CIG-REU/dealii,jperryhouts/dealii,flow123d/dealii,spco/dealii,lue/dealii,sriharisundar/dealii,mac-a/dealii,natashasharma/dealii,YongYang86/dealii,ESeNonFossiIo/dealii,spco/dealii,rrgrove6/dealii,johntfoster/dealii,angelrca/dealii,naliboff/dealii,angelrca/dealii,Arezou-gh/dealii,andreamola/dealii,sairajat/dealii,sairajat/dealii,danshapero/dealii,rrgrove6/dealii,johntfoster/dealii,mac-a/dealii,pesser/dealii,gpitton/dealii,shakirbsm/dealii,flow123d/dealii,shakirbsm/dealii,mtezzele/dealii,johntfoster/dealii,JaeryunYim/dealii,lpolster/dealii,jperryhouts/dealii,angelrca/dealii,pesser/dealii,natashasharma/dealii,pesser/dealii,sairajat/dealii,gpitton/dealii,msteigemann/dealii,ibkim11/dealii,jperryhouts/dealii,nicolacavallini/dealii,lue/dealii,kalj/dealii,naliboff/dealii