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

Pierwsza wersja poprawek #33

Open
wants to merge 22 commits into
base: master
Choose a base branch
from
Open

Pierwsza wersja poprawek #33

wants to merge 22 commits into from

Conversation

matkrak
Copy link

@matkrak matkrak commented Jan 22, 2016

Plik README_RISE.md opisuje instalację RISE (narazie UNIX, jutro Windows)

Poprawki w pierwszym wykładzie + zmienione metadane aby uzyskać lepszy efekt na pokazie slajdów.

Nie często używam githuba, nie wiem gdzie będą widoczne teksty które tu wprowadzam, nie wiem też
w jaki sposób widzi Pan poprawki i czy w zasadzie zrobiłem to tak jak Pan chciał :)

Zmieniona struktura metadanych, w efekcie bardziej przejrzysty pokaz slajdow
Pierwsza wersja instrukcji, narazie UNIX, Windows będzie jutro. Czy taki opis jest wystarczający, czy należy coś dodać/zmienić?
Dodane metadane, kilka miejsc w tym wykładzie wymgaga jeszcze interwencji
git clone https://github.com/damianavila/RISE.git
```

2. Do instalacji należy wykorzystać skrypt `setup.py`, jednak wcześniej trzeba ustawić zmienną<br>
Copy link
Contributor

Choose a reason for hiding this comment

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

Ustawianie czegokolwiek w .bashrc jest uciazliwe dla uzytkownika (moze uzywac innej powloki, moze miec juz taka zmienna ustawiona, moze to zaburzac inne ustawienia w tym pliku). Jesli instalacje da sie przeprowadzic bez modyfikacji .bashrc to byloby swietnie.

Copy link
Author

Choose a reason for hiding this comment

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

ok, już dopisuje alternatywna wersje instalacji bez zmian w bashrc

@grzanka
Copy link
Contributor

grzanka commented Jan 22, 2016

Przydaloby sie umiescic w glownym pliki README.md odnosnik do opisu procesu instalacji.


```
cd RISE/
python setup.py install
Copy link
Contributor

Choose a reason for hiding this comment

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

Prosze dopisac informacje czy ma to byc python2 czy python3. Moze sie tez zdarzyc ze uzytkownik ma obie wersje.

Copy link
Author

Choose a reason for hiding this comment

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

Zrobione, może w wolnej chwili spróbuje zainstalować na pythonie 2.X

Drobne poprawki
Dodano informacje o instalacji RISE.
Dodane metadatane umożliwiające interaktywny pokaz slajdów
Metadane do obsługi pokazu slajdów
Metadane do pokazu slajdów
Metadane do pokazu slajdów. 
Na pewno trzeba zmienić rozmiar obrazków z diagramami - nie mieszczą się na slajdach
a przesuwanie ich jest kłopotliwe
Metadane do pokazu slajdów
@matkrak
Copy link
Author

matkrak commented Jan 23, 2016

Dzień dobry,
Dodałem do wszystkich wykładów metadane.
Żeby wszystko dobrze wyglądało jak napisałem należy zmniejszyć obrazki.

Zajmę się teraz częścią merytoryczną.

Dodano opis instalcji na Windows - jest problem ze zmianą font size (na windows). Pomimo zmiany parametru rozmiar tekstu się nie zmienia
git clone https://github.com/damianavila/RISE.git
```

2. Do instalacji należy wykorzystać skrypt `setup.py`, jednak wcześniej trzeba ustawić zmienną<br>
Copy link
Contributor

Choose a reason for hiding this comment

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

Prosze usunac konfiguracje za pomoca modyfikacji .bashrc

Przedostatnia wersja - jak tylko uda mi sie doprowadzic do porzadku pokaz slajdow bedzie wersja ostateczna
Rozwiazany problem z czcionką, trochę mętny opis <path_to_application> ale sam nie do końca rozumiem po co jest ten katalog i dlaczego właśnie tam. (Rozwiązanie podsunięte przez Damiana Avile). Za chwilę zmienię system i sprawdzę jak wygląda przykładowa ścieżka w przypadku Windowsa.

Cały czas pozostaje problem polskich znaków.
Usunięte komórki wyjściowe
Usunięte komórki wyjściowe
Usunięte komórki wyjściowe
Usunięte komórki wyjściowe
Usunięte komórki wyjściowe
Usunięte komórki wyjściowe
Usunięte komórki wyjściowe
... nieco lepszy opis tego jak znaleść <path_to_application> i dodana przykładowa ścieżka dla Windows. 

Z niewiadomych przyczyn na Windows nie mogę wprowadzić niektórych polskich znaków, poprawię całość na Ubuntu
jak będzie już zatwierdzone (żeby nie robić tego niepotrzebnie)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants