search

paneldata / twinlife

Eine genetisch informative, längsschnittliche Studie zur Entstehung von sozialer Ungleichheit
0 stars 3 forks source link

Update metadata to "v3-0-0" (#40) #44

Closed afuetterer closed 5 years ago

afuetterer commented 5 years ago

Dieser PR updatet die Studie TwinLife aus dem unsauberen Zustand zwischen v2-1 und v-3-0-0 auf die Version v3-0-0, siehe #40. Der PR enthält Lieferungen von @volkerlang, @isieber und @afuetterer. Der PR wird gemergt, sobald die Daten auf dem Testserver begutachtet und für gut befunden wurden. Außerdem soll https://github.com/paneldata/twinlife/pull/43 zuvor gemergt werden.

afuetterer commented 5 years ago

Dieser Stand wird Freitag zunächst auf den Testserver importiert. Ich informiere euch sobald das abgeschlossen ist. Wenn ihr den Stand dann begutachtet habt, würde ich mergen und auf paneldata.org importieren.

Ist das ein guter Plan?

volkerlang commented 5 years ago

Hallo Alex,

ja, klingt nach einem guten Plan.

Danke und viele Grüße, Volker

afuetterer commented 5 years ago

Ich hab euch eine Mail geschickt. Und die Daten vom Branch "v3-0-0" sind hier einzusehen: https://ddionrails.soep.de/twinlife

afuetterer commented 5 years ago

Bitte auf https://ddionrails.soep.de/twinlife nachsehen, ob alles okay ist. Ich würde den PR dann auf den master mergen.

volkerlang commented 5 years ago

Hallo Alex,

danke fürs Einstellen! Wir melden uns, wenn wir mit dem Review fertig sind.

Viele Grüße, Volker

kkrell commented 5 years ago

Danke auch von mir @afuetterer! Mir sind bereits ein paar Dinge aufgefallen. Ich habe dazu neue Kommentare zu drei bereits geschlossenen Issues geschrieben. Ich konnte die leider nicht wieder öffnen. Findest du die Kommentare?

afuetterer commented 5 years ago

Ich finde zwei Kommentare. Wo ist der dritte?

Ich würde das dennoch mergen, damit wir aus dem Provisorium mit dem Branch heraus kommen. Die Änderungen nehmen wir dann mit neuen PRs vor.

kkrell commented 5 years ago

Hallo Alex, #23 ist der dritte. Es geht um die Verknüpfung zwischen Variable und Instrument, die nicht mehr funktioniert. Evtl. ist das dieselbe Fehlerursache wie bei den fehlenden related variables.

Ansonsten können wir es so machen wie du vorschlägst, dass du mergst und wir weitere Änderungen mit neuen PRs vornehmen.