search

Fuenfgeld / ATeamDatenmanagementUndArchivierung

MIT License
7 stars 0 forks source link

Branch felix #28

Closed fphans closed 3 years ago

fphans commented 3 years ago

Header für die importe kann raus

fphans commented 3 years ago

Beim testen liefs durch… daher die idee

Mit freundlichen Grüßen

Felix Hans

UNIVERSITÄTSKLINIKUM FREIBURG Universitäts-Notfallzentrum (UNZ)

Dr. Felix Hans Oberarzt Sir-Hans-A.-Krebs-Strasse 79106 Freiburg Telefon: +49 761 270-33390 Felix.hans@uniklinik-freiburg.de www.uniklinik-freiburg.de/notfallzentrum.html

Von: lknegendorf [mailto:notifications@github.com] Gesendet: Samstag, 23. Januar 2021 12:43 An: Fuenfgeld/ATeamDatenmanagementUndArchivierung Cc: fphans; Mention Betreff: Re: [Fuenfgeld/ATeamDatenmanagementUndArchivierung] Branch felix (#28)

@lknegendorf requested changes on this pull request.

Hey @fphanshttps://github.com/fphans , passt auf jeden Fall so, bis auf die eine Anmerkung

In IMECOS_master.ipynbhttps://github.com/Fuenfgeld/ATeamDatenmanagementUndArchivierung/pull/28#discussion_r563130588:

   },
   "source": [

import requests brauchst du hier zwingend, da du sonst den code aus der .py Datei nicht hier ausführen kannst.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/Fuenfgeld/ATeamDatenmanagementUndArchivierung/pull/28#pullrequestreview-574841077, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASLTOPMYZHDYDOOY6OMKJHDS3KY3JANCNFSM4WPTGJVA.

lknegendorf commented 3 years ago

Beim testen liefs durch… daher die idee Mit freundlichen Grüßen Felix Hans UNIVERSITÄTSKLINIKUM FREIBURG Universitäts-Notfallzentrum (UNZ) Dr. Felix Hans Oberarzt Sir-Hans-A.-Krebs-Strasse 79106 Freiburg Telefon: +49 761 270-33390 Felix.hans@uniklinik-freiburg.de www.uniklinik-freiburg.de/notfallzentrum.html Von: lknegendorf [mailto:notifications@github.com] Gesendet: Samstag, 23. Januar 2021 12:43 An: Fuenfgeld/ATeamDatenmanagementUndArchivierung Cc: fphans; Mention Betreff: Re: [Fuenfgeld/ATeamDatenmanagementUndArchivierung] Branch felix (#28) @lknegendorf requested changes on this pull request. Hey @fphanshttps://github.com/fphans , passt auf jeden Fall so, bis auf die eine Anmerkung ____ In IMECOS_master.ipynb<#28 (comment)>: }, "source": [ - "# import libraries\r\n", - "import sqlite3 as sq\r\n", - "from sqlite3 import Error\r\n", - "import pandas as pd\r\n", - "import numpy as np\r\n", - "import requests\r\n", + "#import libraries (redundant da im 'Update source_database_implementation.py' enthalten und deshalb auskommentiert)\r\n", + "#import sqlite3 as sq\r\n", + "#from sqlite3 import Error\r\n", + "#import pandas as pd\r\n", + "#import numpy as np\r\n", + "#import requests\r\n", import requests brauchst du hier zwingend, da du sonst den code aus der .py Datei nicht hier ausführen kannst. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#28 (review)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASLTOPMYZHDYDOOY6OMKJHDS3KY3JANCNFSM4WPTGJVA.

Vielleicht, weil die imports noch irgendwo im Speicher waren, damit es generisch ist, muss es da hin, weil dieser exec(...-code einen Befehl daraus nutzt.

fphans commented 3 years ago

Check!

Mit freundlichen Grüßen

Felix Hans

UNIVERSITÄTSKLINIKUM FREIBURG Universitäts-Notfallzentrum (UNZ)

Dr. Felix Hans Oberarzt Sir-Hans-A.-Krebs-Strasse 79106 Freiburg Telefon: +49 761 270-33390 Felix.hans@uniklinik-freiburg.de www.uniklinik-freiburg.de/notfallzentrum.html

Von: lknegendorf [mailto:notifications@github.com] Gesendet: Samstag, 23. Januar 2021 12:46 An: Fuenfgeld/ATeamDatenmanagementUndArchivierung Cc: fphans; Mention Betreff: Re: [Fuenfgeld/ATeamDatenmanagementUndArchivierung] Branch felix (#28)

Beim testen liefs durch… daher die idee Mit freundlichen Grüßen Felix Hans UNIVERSITÄTSKLINIKUM FREIBURG Universitäts-Notfallzentrum (UNZ) Dr. Felix Hans Oberarzt Sir-Hans-A.-Krebs-Strasse 79106 Freiburg Telefon: +49 761 270-33390 Felix.hans@uniklinik-freiburg.demailto:Felix.hans@uniklinik-freiburg.de www.uniklinik-freiburg.de/notfallzentrum.htmlhttp://www.uniklinik-freiburg.de/notfallzentrum.html Von: lknegendorf [mailto:notifications@github.commailto:notifications@github.com] Gesendet: Samstag, 23. Januar 2021 12:43 An: Fuenfgeld/ATeamDatenmanagementUndArchivierung Cc: fphans; Mention Betreff: Re: [Fuenfgeld/ATeamDatenmanagementUndArchivierung] Branch felix (#28https://github.com/Fuenfgeld/ATeamDatenmanagementUndArchivierung/pull/28) @lknegendorfhttps://github.com/lknegendorf requested changes on this pull request. Hey @fphanshttps://github.com/fphanshttps://github.com/fphans , passt auf jeden Fall so, bis auf die eine Anmerkung ____ In IMECOS_master.ipynb<#28 (comment)https://github.com/Fuenfgeld/ATeamDatenmanagementUndArchivierung/pull/28#discussion_r563130588>: }, "source": [ - "# import libraries\r\n", - "import sqlite3 as sq\r\n", - "from sqlite3 import Error\r\n", - "import pandas as pd\r\n", - "import numpy as np\r\n", - "import requests\r\n", + "#import libraries (redundant da im 'Update source_database_implementation.py' enthalten und deshalb auskommentiert)\r\n", + "#import sqlite3 as sq\r\n", + "#from sqlite3 import Error\r\n", + "#import pandas as pd\r\n", + "#import numpy as np\r\n", + "#import requests\r\n", import requests brauchst du hier zwingend, da du sonst den code aus der .py Datei nicht hier ausführen kannst. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<#28 (review)https://github.com/Fuenfgeld/ATeamDatenmanagementUndArchivierung/pull/28#pullrequestreview-574841077>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASLTOPMYZHDYDOOY6OMKJHDS3KY3JANCNFSM4WPTGJVA.

Vielleicht, weil die imports noch irgendwo im Speicher waren, damit es generisch ist, muss es da hin, weil dieser exec(...-code einen Befehl daraus nutzt.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/Fuenfgeld/ATeamDatenmanagementUndArchivierung/pull/28#issuecomment-765910823, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASLTOPLF75QIPSYNIIP6PN3S3KZGZANCNFSM4WPTGJVA.