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

MapBuilder Workflow #231

Closed
wants to merge 19 commits into from
Closed

Conversation

pa-ulex
Copy link

@pa-ulex pa-ulex commented Oct 8, 2024

Bezüglich der Issue #186 wollte ich den Mapbuilder zur 2411 Branch hinzufügen.
Die Dateien vom alten Source Mover sind bis auf den Workflow noch drinnen, sonst würden 2 Workflows entstehen.
Es müssten nur noch die Dateien in die Maps/KML Ordner rein und die mapbuilder.toml angepasst werden.
Wie man vielleicht sieht, habe ich mich an dem Bremen Pack inspiriert und nur leicht etwas verändert, da das schon recht gut funktioniert hatte und effizient aus meiner Sicht ist.

@sergej-singer
Copy link
Contributor

sergej-singer commented Oct 13, 2024

ist es möglich eine eigene Version von Mapbuilder reinpacken? Ich habe meins schwer modifiziert, um die Unterstützung von MultiGeometries zu bringen.

@pa-ulex
Copy link
Author

pa-ulex commented Oct 13, 2024

ist es möglich eine eigene Version von Mapbuilder reinpacken?

Ich glaube nicht, da dass mit Docker Packs aus der Original Repo ist, man kann die vielleicht forken und dann so umschreiben das dann die Version von dir genutzt wird.

Edit (letzte Nachricht verbessert):
So, habe mal rumprobiert. Ja, man kann mit einer Fork eigene Sachen hinzufügen und als Docker File hinzufügen. Man muss nur in der Workflow Datei im Mapbuilder Fork die eingene Username reinschreiben, dass gleiche gilt bei diesem Workflow auch oder man macht vielleicht eine EDGG Branch in der offiziellen Repo und nutzt sie dann als geupdatete Version.

@pa-ulex
Copy link
Author

pa-ulex commented Oct 23, 2024

@sergej-singer (@LeoKle)
Ich hätte noch als Idee, dass ich den Mapbuilder hier in die Repo rein und man lässt den die Docker File hier in der Repo generieren mit den Updates von Sergej. So kann man ihn die ganze Zeit umändern, updaten und ihn Custom haben.

@sergej-singer
Copy link
Contributor

Ja, vielleicht ist es am besten. Zurzeit bin ich noch nicht ready alles auf Mapbuilder migrieren, da ich noch mit den EDDF und EDDS ASRs beschäftigt bin.

@pa-ulex pa-ulex changed the base branch from 2411 to main October 23, 2024 15:54
@pa-ulex
Copy link
Author

pa-ulex commented Oct 23, 2024

So, das Mapbuilder Update ist jetzt da und mit den Fixes von @sergej-singer drinnen. Wird jetzt in main gepushed, dann stimmt der Workflow auch. Erst muss der Docker Workflow laufen (ca. 20 bis 30 Min.), dann wenn der ohne Fehler abgelaufen ist, kann der Mapbuilder Ordner geändert werden.

Es wird außerdem nachdem der Docker Workflow hier bei Packages und in VATGER Packages das modifizierte Package angezeigt werden.

@sergej-singer sergej-singer self-assigned this Oct 24, 2024
@pa-ulex pa-ulex closed this Dec 14, 2024
@pa-ulex pa-ulex reopened this Dec 16, 2024
@pa-ulex pa-ulex marked this pull request as draft December 16, 2024 17:19
@pa-ulex pa-ulex marked this pull request as ready for review December 16, 2024 17:19
@pa-ulex pa-ulex changed the base branch from main to Mapbuilder December 16, 2024 17:20
@sergej-singer
Copy link
Contributor

sergej-singer commented Dec 16, 2024

Hi,
ich werde das alles eigentlich selbst schaffen. Ich habe gerade ein Action aus meiner Version von Mapbuilder erstellt und schon in das main Branch integriert. Ich muss das aber noch selbst sorgfältig testen. Ich würde erstmal nur GRP Maps hinzufügen und dann TopSky. Das alles kannst du dich schon im Branch Mapbuilder ansehen.

@pa-ulex pa-ulex closed this Dec 16, 2024
@pa-ulex pa-ulex deleted the mapbuilder-workflow branch December 16, 2024 18:58
@pa-ulex
Copy link
Author

pa-ulex commented Dec 16, 2024

Hi, ich werde das alles eigentlich selbst schaffen. Ich habe gerade ein Action aus meiner Version von Mapbuilder erstellt und schon in das main Branch integriert. Ich muss das aber noch selbst sorgfältig testen. Ich würde erstmal nur GRP Maps hinzufügen und dann TopSky. Das alles kannst du dich schon im Branch Mapbuilder ansehen.

Ja, die Sachen aus der TOML mussten noch geändert werden, die wollte ich noch ändern, aber wenn du das hinbekommst, dann brauchen wir die PR nicht mehr.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants