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 intoPyDataCZ:praha-jaro2020from
messa:202003_pydata_databaze_po_hodine
Open

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

Conversation

@messa
Copy link
Collaborator

@messa messa commented Mar 6, 2020

#82

@review-notebook-app
Copy link

review-notebook-app bot commented Mar 6, 2020

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-jaro2020 March 6, 2020 13:54
@messa
Copy link
Collaborator Author

messa commented Mar 6, 2020

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

@messa messa force-pushed the 202003_pydata_databaze_po_hodine branch from af8e74d to 1b2d547 Compare March 6, 2020 14:03
@messa
Copy link
Collaborator Author

messa commented Mar 6, 2020

Tak teď :) git reflog to the rescue

@messa
Copy link
Collaborator Author

messa commented Mar 6, 2020

Ale ještě to budu upravovat.

@messa messa mentioned this pull request Mar 6, 2020
Merged
frenzymadness
frenzymadness reviewed Mar 12, 2020
View reviewed changes
lessons/pydata/databases/index.ipynb
@@ -115,11 +115,11 @@
"\n",
Copy link
Collaborator

@frenzymadness frenzymadness Mar 12, 2020 •
edited
Loading

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

Copy link
Collaborator Author

@messa messa Mar 13, 2020

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
Copy link
Collaborator

frenzymadness commented Mar 12, 2020

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
Copy link
Collaborator Author

messa commented Mar 13, 2020

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
Copy link
Collaborator Author

messa commented Mar 13, 2020

Líbí se mi to moc.

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

@frenzymadness frenzymadness mentioned this pull request Mar 18, 2020
Merged
@messa messa changed the title WIP: Databáze doplnění Databáze doplnění Mar 18, 2020
janpipek
janpipek reviewed Mar 23, 2020
View reviewed changes
lessons/pydata/databases/index.ipynb
@@ -53,16 +53,16 @@
"\n",
Copy link
Member

@janpipek janpipek Mar 23, 2020

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

lessons/pydata/databases/index.ipynb
@@ -53,16 +53,16 @@
"\n",
Copy link
Member

@janpipek janpipek Mar 23, 2020

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

lessons/pydata/databases/index.ipynb
@@ -53,16 +53,16 @@
"\n",
Copy link
Member

@janpipek janpipek Mar 23, 2020

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
Copy link
Member

janpipek commented Mar 23, 2020

Pěkně to postupuje :-)

@janpipek
Copy link
Member

janpipek commented Sep 18, 2020

Je toto vlastně už hotové? 🤔

@frenzymadness
Copy link
Collaborator

frenzymadness commented Sep 20, 2020

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

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

Reviewers

2 more reviewers

@janpipek janpipek janpipek left review comments

@frenzymadness frenzymadness frenzymadness 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