king-michael / simulation_database

Apache License 2.0

1 stars 1 forks source link

API: function that returns entry + keywords + MetaData in one call #13

Closed king-michael closed 5 years ago

king-michael commented 5 years ago

So we don't have to call three functions but one. If possible in ONE database query.

andrejberg commented 5 years ago

Ist get_entry_details() das was du dir vorgestellt hast?

king-michael commented 5 years ago

Hab sowas quasi damit implementiert, das ich das databaseModel geändert habe, hab es jedoch noch nicht gepusht, weil ich deins durch sehen müsste dafür.

Änderung dahingehend, das

Main.keywords = releationship(..., lazy='dynamic') zu Main.keywords = releationship(..., lazy='joined') geändert wurde. Dadurch besitzt das simulation object direct eine Liste von Keywords und nicht mehr einen query der gefiltet werden konnte und dann mit .all() ausgeführt werden musste.
Desweiteren wurde Main.keywords_query = releationship(lazy='dynamic', viewonly=True) dazu gefügt. Dadurch kann man immer noch den subquery erhalten und nach bedarf filtern.

king-michael commented 5 years ago

Ist get_entry_details() das was du dir vorgestellt hast?

Muss ich testen ob es das macht was ich gerne hätte. Bin mir auch nicht sicher warum du mit d und out arbeitest und nicht nur mit out

andrejberg commented 5 years ago

Du schaust dir glaub ich die falsche funktion an. Auf die Schreibweise kommt es an ;)

Die Funktionen mit großbuchstaben werden entweder umbenannt oder kommen weg.

andrejberg commented 5 years ago

Ich versuche alle "fertigen" Funktionen in /examples/DataBase_API.ipynb zu dokumentieren.

king-michael commented 5 years ago

jetzt! vorhin warn noch last commit 'andrejberg Add group and tag selection to viewer' du hattest sie noch nicht gepusht!

king-michael commented 5 years ago

Du schaust dir glaub ich die falsche funktion an. Auf die Schreibweise kommt es an ;)

Die Funktionen mit großbuchstaben werden entweder umbenannt oder kommen weg.

ja ! wollte schon nen issue aufmachen das die bitte umbenannt werden

king-michael commented 5 years ago

schau mal in #17 für die Änderungen die ich vorschlage.

king-michael commented 5 years ago

response_container_BBPPID{font-family: initial; font-size:initial; color: initial;} Verdammt ich hab vergessen meine solution am Freitag zu pushen. Hab auch eine. Schau mir nachher mal deine an. Von: notifications@github.comGesendet: 10. Dezember 2018 12:34An: simulation_database@noreply.github.comAntworten: reply@reply.github.comCc: michael.king@uni-konstanz.de; author@noreply.github.comBetreff: Re: [king-michael/simulation_database] API: function that returns entry + keywords + MetaData in one call (#13) Closed #13.

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

andrejberg commented 5 years ago

Ich meinte damit eigentlich get_entry_details() aber die ist älter. bestimmt schon 2 wochen alt

king-michael commented 5 years ago

response_container_BBPPID{font-family: initial; font-size:initial; color: initial;} Schau dir mal sim2dict an.Quasi was ähnliches (fehlt noch groups) Von: notifications@github.comGesendet: 10. Dezember 2018 21:25An: simulation_database@noreply.github.comAntworten: reply@reply.github.comCc: michael.king@uni-konstanz.de; author@noreply.github.comBetreff: Re: [king-michael/simulation_database] API: function that returns entry + keywords + MetaData in one call (#13) Ich meinte damit eigentlich get_entry_details() aber die ist älter. bestimmt schon 2 wochen alt

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