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

WIP: Update README.md for higher transparency. #3

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

TomatenMarc
Copy link

@TomatenMarc TomatenMarc commented Nov 29, 2023

I created an updated version of README.md which is ready for review.

This PR solves issue #1 .

@TomatenMarc TomatenMarc added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 29, 2023
@TomatenMarc TomatenMarc self-assigned this Nov 29, 2023
@TomatenMarc TomatenMarc changed the title Update README.md for higher transparency. WIP: Update README.md for higher transparency. Nov 29, 2023
@TomatenMarc TomatenMarc marked this pull request as draft November 29, 2023 12:15
@TomatenMarc TomatenMarc marked this pull request as ready for review November 29, 2023 12:16
Copy link

@MaikeMayer MaikeMayer left a comment

Choose a reason for hiding this comment

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

Der Text gefällt mir sehr gut - schön anschaulich und kleinschnittig. Und ich mag die Emojis. Vielen Dank für die Überarbeitung.
Was mir beim Lesen aufgefallen ist: Im ersten Absatz (Zeilen 13-15) steht "ihr" und später wird die direkte Ansprache via "du" gewählt. Das würde ich vereinheitlichen. Am einfachsten scheint mir, einfach am Anfang ebenfalls die direkte Ansprache über "du" zu verwenden.

README.md Outdated Show resolved Hide resolved
README.md Outdated
[![Intagram](https://img.shields.io/badge/Instagram-%23E4405F.svg?style=for-the-badge&logo=Instagram&logoColor=white)](https://www.instagram.com/heicad_hhu/)
[![Twitter](https://img.shields.io/badge/Twitter-%231DA1F2.svg?style=for-the-badge&logo=Twitter&logoColor=white)](https://twitter.com/HeiCAD_HHU)
<a href="https://ki-campus.org/courses/kifueralle-hhu" style="width: 120px; height: 28px; background-color: #800080; padding: 2px; border-radius: 0px; display: inline-block;">
<img src="https://ki-campus.org/sites/default/files/color/kic_theme-ea158196/logo.svg" alt="Your Badge Alt Text">

Choose a reason for hiding this comment

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

Das Logo ist bis auf "Beta" transparent und durch die weiße Schrift daher nicht sichtbar. Ggf. müssten wir einen (lila) Hintergrund hinzufügen oder ein anderes Bild einbetten.
Noch eine doofe Frage als Laie: Was hat es mit dem alt-Text auf sich?

Copy link
Author

Choose a reason for hiding this comment

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

Der Alt-Text stammt noch aus der Vorlage und wurde entfernt!

Copy link
Author

Choose a reason for hiding this comment

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

@TomatenMarc Todo:

Bezüglich der Darstellung muss ich nochmal überlegen. Im lokalen README wird es korrekt gerendert. Im Browser (GitHub) aber nicht:

Bildschirm­foto 2023-11-30 um 10 24 02

Copy link
Author

Choose a reason for hiding this comment

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

Jetzt stimmt zumindest die Größe, aber die Farbe des Hintergrundes kommt im Browser nicht durch.
Ich schaue da später mal drauf!

Choose a reason for hiding this comment

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

Alles klar, danke dir!

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
## Fertig! :hugs:

Herzlichen Glückwunsch! Du hast nun erfolgreich Anaconda und Git auf deinem Computer installiert.
Wenn du weitere Fragen hast oder Hilfe benötigst, kontaktiere uns. Viel Spaß beim Mitmachen! 😊

Choose a reason for hiding this comment

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

Wollen wir hier ggf. direkt Kontaktdaten verlinken? Oder alternativ einen Textverweis (siehe unten "Kontakt und Unterstützung")

Copy link
Author

Choose a reason for hiding this comment

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

Hatte einfach mal in die Commits geschaut und die ich gefunden hatte angegeben.

Könnte auch alle Kontaktdaten rausgeben, aber da wäre die Frage, ob alle sich mit Code/Git auskennen oder, ob wir nur eine Auswahl an Kontakten hier eintragen wollen?

Choose a reason for hiding this comment

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

Von meiner Seite aus müssen wir nicht zwingend alle Kontaktdaten verlinken. Mir ging es in dem Kommentar eher darum, hier direkt zu sagen, wie die betreffenden Personen erreicht werden können. Aber da reicht im Zweifel auch einfach ein Verweis auf die Kontaktdaten weiter unten im README


## Fertig! :confetti_ball:

Du hast erfolgreich unser Repository heruntergeladen. Wenn du Fragen hast oder Hilfe benötigst, zögere nicht, uns zu kontaktieren! 😊

Choose a reason for hiding this comment

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

siehe Kommentar Zeile 61 (ggf. Kontaktdaten direkt verlinken)

Diese Schritte zeigen, wie du ein Environment namens `AI4all_I` mithilfe der Datei `environment.yml` erstellen und anschließend ein Notebook darin starten kannst.
Es ist wichtig, dass du weißt, wo du das Repository abgelegt hast!

## Schritt 2: Öffnen von Anaconda Navigator :national_park:

Choose a reason for hiding this comment

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

Beim Lesen war ich etwas irritiert, dass hier als erstes "Schritt 2" steht. Ist Schritt 1 abhanden gekommen oder ist bei der Nummerierung vielleicht etwas durcheinander gegangen?

## Fertig! :hugs:

Herzlichen Glückwunsch! Du hast nun erfolgreich Anaconda und Git auf deinem Computer installiert.
Wenn du weitere Fragen hast oder Hilfe benötigst, kontaktiere uns. Viel Spaß beim Mitmachen! 😊
Copy link
Contributor

Choose a reason for hiding this comment

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

Ich habe mich schon gestern gefragt, ob wir auch zu Git-Installation Hilfe anbitten sollen. Das ist zwar lieb, aber ich könnte da zum Beispiel keine Fragen zur Installation auf Windows und Linux beantworten, weil ich diese Betriebsysteme nicht habe.

Copy link
Author

Choose a reason for hiding this comment

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

Das sollte soweit eigentlich kein Problem sein, man wählt auf der Seite eigentlich direkt das jeweilige System aus und folgt der Anleitung. Da könnte ich aber zur Not helfen, wenn Fragen aufkommen!

Copy link
Contributor

Choose a reason for hiding this comment

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

Es geht hier ja darum, diese Dateien unabhängig von unserem Projekt zur Verfügung zu stellen. In zwei Jahren ist bei uns (wahrscheinlich) niemand mehr erreichbar.

README.md Outdated

## Herunterladen über HTTPS :bouquet:

Folge diesen Schritten, um das Repository über HTTPS zu klonen:
Copy link
Contributor

Choose a reason for hiding this comment

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

Ich frage mich, ob der Begriff "klonen" auch für normale Menschen verständlich ist bzw. verständlich ist, was da passieren wird. Sonst könnte man es z.B. durch "kopieren der Dateien auf deinen Computer" ersetzen.

Copy link
Author

Choose a reason for hiding this comment

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

Wurde geändert!

Viel Spaß beim Programmieren! 😊

# Kontakt und Unterstützung :handshake:

Copy link
Contributor

Choose a reason for hiding this comment

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

Ich habe mich gerade gefragt, ob wir zu Punkten 3, 4 und 5 vielleicht die entsprechenden Links zu Anaconda Manual verlinken dürfen. Es gibt Menschen wie z.B. mich ;), die es selber hinbekommen wollen, bevor sie andere kontaktieren.

Copy link
Author

Choose a reason for hiding this comment

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

Wir verlinken auf die Anaconda-Webseite. Da sollte man auch direkt die Doku finden :-)

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Co-authored-by: MaikeMayer <112072849+MaikeMayer@users.noreply.github.com>
Copy link
Contributor

@AKSelker AKSelker left a comment

Choose a reason for hiding this comment

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

Finde ich gut, würde nur noch mal über die Ansprechpartner reden wollen, bevor wir das so hochladen.

Willkommen in unserem Repository für den ersten [KI für Alle](https://ki-campus.org/courses/kifueralle-hhu)-Kurs!

Hier findest du alle Jupyter Notebooks zu den Programmierteilen aus dem Kurs für die verschiedenen Wochenabschnitte.
Stell sicher, dass du den Anweisungen folgt, damit du problemlos auf deinem eigenen Computer arbeiten kannst.
Copy link
Contributor

Choose a reason for hiding this comment

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

folgt -> folgst

## Fertig! :hugs:

Herzlichen Glückwunsch! Du hast nun erfolgreich Anaconda und Git auf deinem Computer installiert.
Wenn du weitere Fragen hast oder Hilfe benötigst, kontaktiere uns. Viel Spaß beim Mitmachen! 😊
Copy link
Contributor

Choose a reason for hiding this comment

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

Es geht hier ja darum, diese Dateien unabhängig von unserem Projekt zur Verfügung zu stellen. In zwei Jahren ist bei uns (wahrscheinlich) niemand mehr erreichbar.


Wenn du das Repository lieber als ZIP-Datei herunterladen möchtest, folge diesen Schritten:

1. Gehe zu [GitHub](https://github.com/HeiCAD/AI4all_I/).
Copy link
Contributor

Choose a reason for hiding this comment

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

Finde die Formulierung irreführend: "Gehe zu Github", der Link führt aber zu unserem Repository.

# Erstellen eines Environment mit Anaconda Navigator :globe_with_meridians:

Diese Schritte zeigen, wie du ein Environment namens `AI4all_I` mithilfe der Datei `environment.yml` erstellen und anschließend ein Notebook darin starten kannst.
Es ist wichtig, dass du weißt, wo du das Repository abgelegt hast!
Copy link
Contributor

Choose a reason for hiding this comment

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

Dieser Kommentar sollte vielleicht auch schon weiter oben beim Herunterladen stehen, meine Eltern speichern auch immer ständig was und finden es dann nicht wieder.


Starte die `Anaconda Navigator`-Anwendung auf deinem Computer.

## Schritt 3: Erstellen des Environments :nut_and_bolt:
Copy link
Contributor

Choose a reason for hiding this comment

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

Die Unterpunkte in diesem Schritt beziehen sich alle auf die aktuelle Anaconda-Version. Bei neuen Versionen kann sich links, unten etc ändern. Wir sollten hier vielleicht noch die Anacondaversion angeben, in der diese Anweisungen auf jeden Fall funktionieren, und dann regelmäßig (jedes Semester) aktualisieren. Oder eher mit Screenshots arbeiten?


Wir schätzen deine Beteiligung und stehen bereit, um sicherzustellen, dass dein Beitrag erfolgreich ist! 😊

Beep Boop: Dein HeiCAD-Team :robot:
Copy link
Contributor

Choose a reason for hiding this comment

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

Hier vielleicht noch die Heicad-Seite verlinken? Wer weiß, wie lange die oben genannten Ansprechpartner bleiben, auf der Heicad-Seite wird es immer aktuelle Ansprechpartner zu finden geben.


Falls du Fragen hast, Probleme auftreten oder du einfach nur mit uns in Kontakt treten möchtest, stehen wir dir gerne zur Verfügung. Hier sind verschiedene Möglichkeiten, wie du uns erreichen kannst:

## Mitglieder des Teams: :alien: :vulcan_salute:
Copy link
Contributor

Choose a reason for hiding this comment

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

Ich habe ein Problem damit, hier spezifische Namen als Ansprechpartner zu geben. Nachdem Joana gegangen war, musste sie ständig noch Dinge weiterleiten, die an ihre Emailadresse gingen. Besser die Funktionsemailadresse angeben, da sitzt immer jemand hinter und kann die Mails an die entsprechenden Personen weiterleiten.


Willkommen in unserem Repository für den ersten [KI für Alle](https://ki-campus.org/courses/kifueralle-hhu)-Kurs!

Hier findest du alle Jupyter Notebooks zu den Programmierteilen aus dem Kurs für die verschiedenen Wochenabschnitte.

Choose a reason for hiding this comment

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

Wir sollten hier sicherheitshalber noch einen Hinweis auf die Lizenz ergänzen, unter der das Material zur Verfügung steht. Ich glaube, bisher steht das nirgends auf Git.
Beispiel: "Das Material steht unter CC-BY 4.0 zur Verfügung." -> https://creativecommons.org/licenses/by/4.0/deed.de

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants