venv activation

[AlvinSchiller]

POC für eine mögliche Strukurierung. Bisher keinerlei Test der angepassten skripte!
Wie im Kommentar hier beschrieben MiczFlor#2144 (comment)

• run*.py skripte in die fachlichen Ordner verschoben. Vorerst rückgängig gemacht
  • Falls man die run skripte in unterordner haben will, müsste man den python import anpassen, sodass die gesamte struktur verfügbar ist. Eine möglichkeit wäre z.B. eine installation mit pip (https://stackoverflow.com/a/50194143). Damit müssten aber evtl alle imports angepasst werden.
• shell skripte für die Aufrufe der python skripte inkl venv + workingdir handling hinzugefügt.
  • platzierung in den fachlichen Bereichen z.B.
    /installation/components für rfid reader und audio
    /tools für rpc und sniffer
    ./ für jukebox

[coveralls]

Pull Request Test Coverage Report for Build 7738986943

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage remained the same at 38.384%

---

Totals
Change from base Build 7717240653:	0.0%
Covered Lines:	418
Relevant Lines:	1089

---

💛 - Coveralls

[pabera]

Progressive change, but I like the idea a lot! +1
This change requires a proper update mechanism, though. Otherwise you won't be able to execute this. So maybe this is not something for v3.5

[pabera]

I see this code a few times. Could it be abstracted?

[AlvinSchiller]

not very much sadly.
If we wanted to source another script, we still need the calculation for the project root, to access it.
And after that its only the venv activation. This could be extracted if we bother, to make future changes easier, but don't know if its worth.

[AlvinSchiller]

Progressive change, but I like the idea a lot! +1 This change requires a proper update mechanism, though. Otherwise you won't be able to execute this. So maybe this is not something for v3.5

Yes, i would also see this for the v3.6.0 release.

As you state it, i think few have a problem with the current update steps already for v3.5.0 -> MiczFlor#2211 (comment)

[AlvinSchiller]

Warum eigentlich englisch... ist ja ein PR auf meinem Repo :D

Wenn ihr sagt das Vorgehen passt und das sollten wir weiter verfolgen, würde ich die Änderungen auf den ursprünglichen PR MiczFlor#2144 commiten, damit weitere Kommentare da getrackt werden und erhalten bleiben.
Das hier war erstmal nur als POC damit ihr es euch ansehen könnt

[AlvinSchiller]

Wie seht ihr übrigends den Punkt? Sollte man es ermöglichen die Run Skripte auch in den Unterordner zu haben? Grundsätzlich wäre das wohl erstrebenswert, um ne klare struktur zu haben, und evtl auch mehr in python skripten zu entwickeln. Wäre natürlich schon etwas aufwand, und ich bin da in python auch nicht tief drin, um das genau zu bewerten.

• run*.py skripte in die fachlichen Ordner verschoben. Vorerst rückgängig gemacht
  • Falls man die run skripte in unterordner haben will, müsste man den python import anpassen, sodass die gesamte struktur verfügbar ist. Eine möglichkeit wäre z.B. eine installation mit pip (https://stackoverflow.com/a/50194143). Damit müssten aber evtl alle imports angepasst werden.

[s-martin]

Wie seht ihr übrigends den Punkt? Sollte man es ermöglichen die Run Skripte auch in den Unterordner zu haben? Grundsätzlich wäre das wohl erstrebenswert, um ne klare struktur zu haben, und evtl auch mehr in python skripten zu entwickeln. Wäre natürlich schon etwas aufwand, und ich bin da in python auch nicht tief drin, um das genau zu bewerten.

• run*.py skripte in die fachlichen Ordner verschoben. Vorerst rückgängig gemacht

  • Falls man die run skripte in unterordner haben will, müsste man den python import anpassen, sodass die gesamte struktur verfügbar ist. Eine möglichkeit wäre z.B. eine installation mit pip (https://stackoverflow.com/a/50194143). Damit müssten aber evtl alle imports angepasst werden.

Wäre sicher sauberer strukturiert, aber ich denke das könnte man auch noch später machen, falls wir jetzt noch keine Lösung dafür haben

[AlvinSchiller]

closed
PR für main repo erstellt -> MiczFlor#2233