search

mathLab / pi-DoMUS

Parallel Deal.II MUltiphysics Solver
http://mathlab.github.io/pi-DoMUS/
GNU Lesser General Public License v2.1
16 stars 16 forks source link

Different mapping for different purpose #201

Closed ESeNonFossiIo closed 8 years ago

ESeNonFossiIo commented 8 years ago

I would like to use this PR for ALE in pi-DoMUS. In particular I need to attach a signal to each get_###_mapping. Signals are used in pidomus.cc and not in _baseinterface.cc. @asartori86 ideas?

luca-heltai commented 8 years ago

Ovunque:

Instead of 

This function is called...

Use

Return the mapping used to...
luca-heltai commented 8 years ago

All methods comment should be in the imperative tense...

Do this

Do that

Return this

Return that

ESeNonFossiIo commented 8 years ago

Ok... I have copied the old comment. Once i write a working code i will fix all comments. ;)

Mauro

Il giorno 29 giu 2016, alle ore 09:34, Luca Heltai notifications@github.com ha scritto:

All methods comment should be in the imperative tense...

Do this

Do that

Return this

Return that

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

ESeNonFossiIo commented 8 years ago

Come scrivo i signals senza incasinare il codice? Faccio una classe pidomus-mapping con un elemento signal?

Mauro

Il giorno 29 giu 2016, alle ore 09:34, Luca Heltai notifications@github.com ha scritto:

All methods comment should be in the imperative tense...

Do this

Do that

Return this

Return that

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub, or mute the thread.

asartori86 commented 8 years ago

Come scrivo i signals senza incasinare il codice? Faccio una classe pidomus-mapping con un elemento signal?

perché hai bisogno dei signal? sono già funzioni virtuali, quindi puoi fargli fare quello che vuoi no?

ESeNonFossiIo commented 8 years ago

devo aggiornarle ogni volta che calcolo il residuo…

On 29 Jun 2016, at 10:07, Alberto Sartori notifications@github.com wrote:

Come scrivo i signals senza incasinare il codice? Faccio una classe pidomus-mapping con un elemento signal?

perché hai bisogno dei signal? sono già funzioni virtuali, quindi puoi fargli fare quello che vuoi no?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mathLab/pi-DoMUS/pull/201#issuecomment-229285973, or mute the thread https://github.com/notifications/unsubscribe/AIisRbTaIMZwfIQXrcCHOpMdLurrpZpTks5qQiezgaJpZM4JAWGm.

luca-heltai commented 8 years ago

Non c'e' bisogno di aggiornare niente. MappingQEulerian si prende una referenza al vettore con cui lo hai costruito, e si memorizza internamente il vettore. Se cambia la soluzione, cambia il mapping. Devi aggiornarlo ogni volta che raffini, questo si.

ESeNonFossiIo commented 8 years ago

@luca-heltai update dei commenti

ESeNonFossiIo commented 8 years ago

Comments updated

luca-heltai commented 8 years ago

Ok. Merge if it passes.