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

Setup scientific simulation new Tutorial #1086

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

Conversation

Kimneyyy
Copy link

No description provided.

@svenja11 svenja11 added the review wanted Request a review label Feb 17, 2025
@svenja11
Copy link
Collaborator

Vielen Dank für das Tutorial @Kimneyyy. Ich habe es mir angesehen und ein paar Kommentare.

@svenja11
Copy link
Collaborator

Ich habe die Formatierung des deutschen Tutorials angepasst, damit es einheitlich ist mit den bestehenden Tutorials und damit es etwas leichter ist diesem zu folgen. Könnt ihr die englische Version bitte entsprechend anpassen? Siehe dazu: Tutorial Template

@svenja11
Copy link
Collaborator

Ich war etwas überrascht, dass man auf zwei verschiedenen Geräten arbeitet, jedoch keine Daten vom Ubuntu-Server auf das Windows-Gerät kopiert werden. Ist das so richtig?

@svenja11
Copy link
Collaborator

Wäre es möglich in Schritt 5.1 noch Befehle für PowerShell zu ergänzen? Aktuell wird nur gezeigt, wie man AGRender mit einem WSL-Subsystem unter Windows klont.

@svenja11
Copy link
Collaborator

svenja11 commented Feb 20, 2025

Ich habe versucht dem Tutorial selbst Schritt-für-Schritt zu folgen und bin an dem Punkt "Schritt 5.1" nicht mehr weitergekommen.

Die interaktive Umgebung startet noch wie erwartet. Ich bin mir nur nicht ganz sicher was an der Stelle "Choose a data folder to load" erwartet wird. Wenn ich einfach nur einen Ordner angebe, wie z.B. output_data, wird die interaktive Umgebung automatisch beendet mit der Fehlermeldung "Invalid input". Im Tutorial steht nur "Wähle den outputfolder aus".
Viel mehr Optionen habe ich allerdings nicht, weil auf dem Windows-Server ja nur diese Ordner liegen:

<parent-directory>/
├── output_data
└── AGRender

Alle anderen Daten liegen auf dem Ubuntu-Server.

Könnt ihr diesen Schritt bitte noch etwas ausbauen?

@svenja11
Copy link
Collaborator

Bitte ergänzt noch ein Ergebnis bzw. im Englischen eine Conclusion.

@svenja11
Copy link
Collaborator

Bitte fügt am Ende von 01.de.md und 01.en.md noch diesen License-Block ein und ersetzt [submitter's name and email address here] mit echten Daten:



Bitte fügt zusätzlich auch diesen Kommentar in der Beschreibung dieses PRs ein und ersetzt auch dort YOUR NAME <[email protected]> mit echten Daten:

I have read and understood the Contributor's Certificate of Origin available at the end of
https://raw.githubusercontent.com/hetzneronline/community-content/master/tutorial-template.md
and I hereby certify that I meet the contribution criteria described in it.
Signed-off-by: YOUR NAME [email protected]

@svenja11 svenja11 added needs action Something has to be updated and removed review wanted Request a review labels Feb 20, 2025
@Kimneyyy
Copy link
Author

Wäre es möglich in Schritt 5.1 noch Befehle für PowerShell zu ergänzen? Aktuell wird nur gezeigt, wie man AGRender mit einem WSL-Subsystem unter Windows klont.

Der Befehl ist für beide Systeme gleich und es steht nicht explizit dran, dass man ein wsl sub-system verwenden muss, oder?

@Kimneyyy
Copy link
Author

Ich war etwas überrascht, dass man auf zwei verschiedenen Geräten arbeitet, jedoch keine Daten vom Ubuntu-Server auf das Windows-Gerät kopiert werden. Ist das so richtig?

Ich hbe einen weiteren Punkt hinzugefügt. Ist das so ok oder übersieht man das zu schnell?

@Kimneyyy
Copy link
Author

Ich habe versucht dem Tutorial selbst Schritt-für-Schritt zu folgen und bin an dem Punkt "Schritt 5.1" nicht mehr weitergekommen.

Die interaktive Umgebung startet noch wie erwartet. Ich bin mir nur nicht ganz sicher was an der Stelle "Choose a data folder to load" erwartet wird. Wenn ich einfach nur einen Ordner angebe, wie z.B. output_data, wird die interaktive Umgebung automatisch beendet mit der Fehlermeldung "Invalid input". Im Tutorial steht nur "Wähle den outputfolder aus". Viel mehr Optionen habe ich allerdings nicht, weil auf dem Windows-Server ja nur diese Ordner liegen:

<parent-directory>/
├── output_data
└── AGRender

Alle anderen Daten liegen auf dem Ubuntu-Server.

Könnt ihr diesen Schritt bitte noch etwas ausbauen?

Der Fehler tritt auf, weil keine berechnetten Daten zum anzeigen vorhanden sind.
Hat sich dieser Punkt geklärt, weil ich dazu geschrieben habe, dass die Daten vom ubuntu server auf den Windows computer kopiert werden müssen.

@Kimneyyy
Copy link
Author

Bitte fügt am Ende von 01.de.md und 01.en.md noch diesen License-Block ein und ersetzt [submitter's name and email address here] mit echten Daten:

* [License block](https://github.com/hetzneronline/community-content/blob/master/tutorial-template.md?plain=1#L108C1-L137C4)

Bitte fügt zusätzlich auch diesen Kommentar in der Beschreibung dieses PRs ein und ersetzt auch dort YOUR NAME <[email protected]> mit echten Daten:

I have read and understood the Contributor's Certificate of Origin available at the end of
https://raw.githubusercontent.com/hetzneronline/community-content/master/tutorial-template.md
and I hereby certify that I meet the contribution criteria described in it.
Signed-off-by: YOUR NAME [email protected]

Ist das so richtig, wie ich es ganz oben eingefügt habe?

@svenja11
Copy link
Collaborator

Ich hbe einen weiteren Punkt hinzugefügt. Ist das so ok oder übersieht man das zu schnell?

Bitte sei etwas präziser und erkläre, wo der Ordner auf dem Server liegt und wo der Ordner auf dem Windows-Gerät hin muss. Zum Beispiel anhand eines Directory-Trees:

<parent-directory>/
├── output_data
└── AGRender
    └── pre_build
        └── type.exe

Bitte ergänze auch den entsprechenden Befehl. Zum Beispiel:

scp -r <user>@<server_ip>:~/AstroGen2/output_data <local-path>

Bitte erklärt auch, welchen Pfad man genau angeben muss bei "Choose a data folder to load", etwa anhand dieses Beispiels:

<parent-directory>/
├── output
└── AGRender
    └── pre_build
        └── type.exe

Ist der Pfad dann relativ zu type.exe? Also:

Choose a data folder to load:

../../output

Oder muss der Output im selben Ordner sein wie type.exe selbst?

<parent-directory>/
└── AGRender
    └── pre_build
        ├── output
        └── type.exe
Choose a data folder to load:

output

Bei den Tutorials ist es wichtig, dass jeder einzelne Schritt aufgeführt und erklärt wird. Im besten Fall sollte nichts als selbstverständlich angesehen werden, damit jeder diesem Tutorial folgen hat.

@svenja11
Copy link
Collaborator

Danke fürs Hinzufügen des Licence Blocks. Der ist an der richtigen Stelle.

Bitte entferne aber wieder den Kommentar aus dem Tutorial. Der gehört nicht ins Tutorial, sondern in den PR. Füge den Kommentar bitte als Beschreibung in diesem PR ein.

@svenja11
Copy link
Collaborator

Bitte denkt auch noch daran ein Ergebnis bzw. im Englischen eine Conclusion zu ergänzen.

@Kimneyyy
Copy link
Author

Ich hbe einen weiteren Punkt hinzugefügt. Ist das so ok oder übersieht man das zu schnell?

Bitte sei etwas präziser und erkläre, wo der Ordner auf dem Server liegt und wo der Ordner auf dem Windows-Gerät hin muss. Zum Beispiel anhand eines Directory-Trees:

<parent-directory>/
├── output_data
└── AGRender
    └── pre_build
        └── type.exe

Bitte ergänze auch den entsprechenden Befehl. Zum Beispiel:

scp -r <user>@<server_ip>:~/AstroGen2/output_data <local-path>

Bitte erklärt auch, welchen Pfad man genau angeben muss bei "Choose a data folder to load", etwa anhand dieses Beispiels:

<parent-directory>/
├── output
└── AGRender
    └── pre_build
        └── type.exe

Ist der Pfad dann relativ zu type.exe? Also:

Choose a data folder to load:

../../output

Oder muss der Output im selben Ordner sein wie type.exe selbst?

<parent-directory>/
└── AGRender
    └── pre_build
        ├── output
        └── type.exe
Choose a data folder to load:

output

Bei den Tutorials ist es wichtig, dass jeder einzelne Schritt aufgeführt und erklärt wird. Im besten Fall sollte nichts als selbstverständlich angesehen werden, damit jeder diesem Tutorial folgen hat.

Ich habe diesen Part überarbeitet. Ist es jetzt verständlich und nachvolyiehbar?

@Kimneyyy
Copy link
Author

Bitte denkt auch noch daran ein Ergebnis bzw. im Englischen eine Conclusion zu ergänzen.

Ist das Ergebnis in dieser Ausführung ok?

@svenja11
Copy link
Collaborator

Bitte fügt noch diesen Kommentar als Beschreibung für diesen PR ein und ersetzt YOUR NAME <[email protected]> mit echten Daten:

I have read and understood the Contributor's Certificate of Origin available at the end of
https://raw.githubusercontent.com/hetzneronline/community-content/master/tutorial-template.md
and I hereby certify that I meet the contribution criteria described in it.
Signed-off-by: YOUR NAME [email protected]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs action Something has to be updated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants