Skip to content

Conversation

@matildeRivas
Copy link
Contributor

context
We have multiple plugins and tools that require the latest version of qgis or a stable operating system.

problem
Not all of our users can install the latest qgis version or have supported operating systems (e.g. < MacOS13, testing Linux distributions) so they cannot use our tools.

proposed solution
A container that is built on a stable Linux OS, installs the latest QGIS along with all of our tools. When the container is run, it opens the QGIS desktop application with Fire2A plugins installed.

@matildeRivas matildeRivas requested a review from fdobad March 7, 2025 14:48
@matildeRivas matildeRivas self-assigned this Mar 7, 2025
Copy link
Member

@fdobad fdobad left a comment

Choose a reason for hiding this comment

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

readme for devs, the least the better. maybe even a script.sh

Dockerfile-qgis Outdated
RUN mkdir -p /root/.local/share/QGIS/QGIS3/profiles/default/python/plugins/ && \
ln -s toolbox/fireanalyticstoolbox /root/.local/share/QGIS/QGIS3/profiles/default/python/plugins/ && \
cd toolbox/fireanalyticstoolbox/simulator && \
ln -s C2F . && \
Copy link
Member

Choose a reason for hiding this comment

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

ln -s /mnt/C2F-W C2F ?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Dockerfile-qgis Outdated
pip install -r requirements.txt && \
pip install --editable .


Copy link
Member

Choose a reason for hiding this comment

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

optional improvement:

USER fire2a ?

objective: avoid the root account in the next steps

Copy link
Contributor Author

Choose a reason for hiding this comment

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

por qué?

@matildeRivas
Copy link
Contributor Author

readme for devs, the least the better. maybe even a script.sh

el TL;DR no es suficiente?

@fdobad fdobad merged commit 6b4c21b into main Mar 12, 2025
1 of 2 checks passed
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.

3 participants