Skip to content
This repository was archived by the owner on Sep 11, 2024. It is now read-only.

Databáze doplnění - #83

Open
messa wants to merge 9 commits into
PyDataCZ:praha-jaro2020from
messa:202003_pydata_databaze_po_hodine
Open

Databáze doplnění#83
messa wants to merge 9 commits into
PyDataCZ:praha-jaro2020from
messa:202003_pydata_databaze_po_hodine

Conversation

@messa

@messamessa commented Mar 6, 2020

Copy link
Copy Markdown
Collaborator

@review-notebook-app

review-notebook-appBot commented Mar 6, 2020

Copy link
Copy Markdown

Check out this pull request on ReviewNB

You'll be able to see Jupyter notebook diff and discuss changes. Powered by ReviewNB.

@messa
messa changed the base branch from master to praha-jaro2020March 6, 2020 13:54
@messa

messa commented Mar 6, 2020

Copy link
Copy Markdown
CollaboratorAuthor

Tak teď se mi asi díky nějakému git fu povedlo smazat moje úpravy...

@messa
messaforce-pushed the 202003_pydata_databaze_po_hodine branch from af8e74d to 1b2d547CompareMarch 6, 2020 14:03
@messa

messa commented Mar 6, 2020

Copy link
Copy Markdown
CollaboratorAuthor

Tak teď :) git reflog to the rescue

@messa

messa commented Mar 6, 2020

Copy link
Copy Markdown
CollaboratorAuthor

Ale ještě to budu upravovat.

@messamessa mentioned this pull request Mar 6, 2020
Merged
frenzymadness
frenzymadness reviewed Mar 12, 2020
View reviewed changes
Comment threadlessons/pydata/databases/index.ipynb
@@ -115,11 +115,11 @@
"\n",

@frenzymadnessfrenzymadnessMar 12, 2020•
edited
Loading

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Vyměnil bych security → bezpečnost když je všechno ostatní moc hezky česky


Reply via ReviewNB

@messamessaMar 13, 2020

Copy link
Copy Markdown
CollaboratorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Přijde mi, že "bezpečnost" je širší pojem, než "security". No ale celkově bych to napsal asi jinak, už ani nevím, co jsem tím přesně myslel :) Zkusím to upravit.

@frenzymadness

frenzymadness commented Mar 12, 2020

Copy link
Copy Markdown
Collaborator

Líbí se mi to moc. Nevím, jestli už třeba HAVING není moc, ale beru to tak, že se to bude ukazovat jedno po druhém od snadných věcí po ty složitější a když se něco nestihne vysvětlit, tak si to dojedou doma. 👍

@messa

messa commented Mar 13, 2020

Copy link
Copy Markdown
CollaboratorAuthor

Nevím, jestli už třeba HAVING není moc

Existuje spousta tutorialů na SQL. V kurzu jsem se snažil předat informace, které jsou zásadní a nejsou v těch tutorialech úplně vidět. Tady jde o strukturu a provádění SELECTu. Chtěl jsem tím říct, že SELECT může mít spoustu částí a každá má svůj specifický význam - že WHERE filtruje řádky načtené z tabulky (tabulek), pak se to seskupuje podle GROUP BY, pak tohle celé můžu filtrovat přes HAVING, a pak zase tohle celé sortuju a limituju. Aby v tom neměli zmatek a nedělali blbosti typu "když to nefunguje ve WHERE, zkusím to dát do HAVING", ale aby pochopili, co se děje. A zrovna u cílovky PyData kurzu bych užívání agregací (GROUP BY) očekával.

Koneckonců, možnosti SELECT tím zdaleka nekončí a vynechal jsem toho spoustu :) Vynechal jsem třeba úplně subquery, ale to je zrovna věc, u které bych řekl, že je celkem pochopitelná při samostudiu.

@messa

messa commented Mar 13, 2020

Copy link
Copy Markdown
CollaboratorAuthor

Líbí se mi to moc.

Super, díky :) Tak snad tento víkend materiály dotáhnu 😅

@frenzymadnessfrenzymadness mentioned this pull request Mar 18, 2020
Merged
@messamessa changed the title WIP: Databáze doplněníDatabáze doplněníMar 18, 2020
janpipek
janpipek reviewed Mar 23, 2020
View reviewed changes
Comment threadlessons/pydata/databases/index.ipynb
@@ -53,16 +53,16 @@
"\n",

@janpipekjanpipekMar 23, 2020

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Teď nechci být rejpal, ale pak by možná bylo elegantní to na začátku vyměnit.


Reply via ReviewNB

Comment threadlessons/pydata/databases/index.ipynb
@@ -53,16 +53,16 @@
"\n",

@janpipekjanpipekMar 23, 2020

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

...rekapitulace, jak se probíhá


Reply via ReviewNB

Comment threadlessons/pydata/databases/index.ipynb
@@ -53,16 +53,16 @@
"\n",

@janpipekjanpipekMar 23, 2020

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tohle je fakt nesympatickej error - to mrví sqlalchemy?


Reply via ReviewNB

@janpipek

janpipek commented Mar 23, 2020

Copy link
Copy Markdown
Member

Pěkně to postupuje :-)

@janpipek

janpipek commented Sep 18, 2020

Copy link
Copy Markdown
Member

Je toto vlastně už hotové? 🤔

@frenzymadness

frenzymadness commented Sep 20, 2020

Copy link
Copy Markdown
Collaborator

V aktuálním pražském kurzu jsou na konci této lekce nadpisy bez obsahu, takže asi není.

Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

2 more reviewers
@janpipekjanpipekjanpipek left review comments
@frenzymadnessfrenzymadnessfrenzymadness left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@messa@frenzymadness@janpipek