Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Change homeworks to Labelord #91

Merged
merged 7 commits into from
Sep 26, 2017
Merged

Change homeworks to Labelord #91

merged 7 commits into from
Sep 26, 2017

Conversation

MarekSuchanek
Copy link
Collaborator

New homework series branch for review... Still working on Labelord II (flask) - will be done soon.

@hroncok
Copy link
Member

hroncok commented Sep 26, 2017

Ad termíny: Vždy konec pondělí za 12 dní.

@MarekSuchanek
Copy link
Collaborator Author

A chceme termíny explicitně v každém zadání nebo jenom řekneme na cvičení/napíšeme do 00_uvod.md?

@MarekSuchanek
Copy link
Collaborator Author

A asi by to chtělo ještě rozhodit, aby flask úkol nebyl hned po requests+click ...

@hroncok
Copy link
Member

hroncok commented Sep 26, 2017

A chceme termíny explicitně v každém zadání nebo jenom řekneme na cvičení/napíšeme do 00_uvod.md?

Napsal bych do úvodu ten deadline na úkoly se závorkou (pokud není přímo u úkolu uvedeno jinak).

A asi by to chtělo ještě rozhodit, aby flask úkol nebyl hned po requests+click ...

Aha. Potom se zdá, že něco je špatně. Podle #79 (comment) jsme se dohodli, že na první úkol bude méně času. Otázka je, jestli to je doable. Nerad bych teď šíbroval s pořadím.

@hroncok
Copy link
Member

hroncok commented Sep 26, 2017

Navrhuji teda udělat to tak, že první dva úkoly budou mít společný deadline 12 dní po cvičení na Flask. Znamená to, že studenti nemohou využít naše řešení úlohy na requests pro úlohu na Flask, ale přijde mi to jako lepší řešení, než jim dát na (ne zas tak jednoduchou) úlohu termín kratší než týden.

@hroncok
Copy link
Member

hroncok commented Sep 26, 2017

@MarekSuchanek rád bych tohle v průběhu dneška, případně zítra do 16:00 mergnul. Můžeš prosím vyřešit ta TODO v první úloze?

@@ -13,64 +13,243 @@ Výukové materiály o clicku:
[naucse.python.cz](http://naucse.python.cz/2017/mipyt-zima/intro/click/),
[GitHub](https://github.com/pyvec/naucse.python.cz/tree/master/lessons/intro/click).

Úkol
----
Úkol - Labelord I
Copy link
Member

Choose a reason for hiding this comment

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

Můžeme to nechat jen jako Úkol, ať nemusím měnit odkazy s kotvou? Je to prkotina, ale zbytečná pakárna. Díky

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Jasně, hotovo...

@MarekSuchanek
Copy link
Collaborator Author

Navrhuji teda udělat to tak, že první dva úkoly budou mít společný deadline 12 dní po cvičení na Flask. Znamená to, že studenti nemohou využít naše řešení úlohy na requests pro úlohu na Flask, ale přijde mi to jako lepší řešení, než jim dát na (ne zas tak jednoduchou) úlohu termín kratší než týden.

Ono to zadání je sice dlouhé (kvůli tomu, že je tam plno možností, co se může stát, a že to musí mít všichni stejné) ale za ten 1 týden by to neměl být zase problém... Nebo bych to mohl ještě ořezat (například jenom verbose styl výpisu a/nebo vyhodit dry-run). Promiň za zmatek.

@MarekSuchanek rád bych tohle v průběhu dneška, případně zítra do 16:00 mergnul. Můžeš prosím vyřešit ta TODO v první úloze?

TODO už tam nejsou...

Copy link
Member

@hroncok hroncok left a comment

Choose a reason for hiding this comment

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

Na beta verzi pro KAM good enough. Tam se odladí detaily.

@hroncok hroncok changed the title WIP: Change homeworks to Labelord Change homeworks to Labelord Sep 26, 2017
@hroncok hroncok merged commit c1f1e19 into master Sep 26, 2017
@kalvotom
Copy link
Contributor

kalvotom commented Oct 1, 2017

@hroncok Good enough, jo? Tak to jsem zvědav.

@MarekSuchanek MarekSuchanek deleted the labelord branch October 2, 2017 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants