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

publish r.dop.import module #3

Merged
merged 111 commits into from
Aug 21, 2024
Merged

publish r.dop.import module #3

merged 111 commits into from
Aug 21, 2024

Conversation

JHalbauer
Copy link
Collaborator

@JHalbauer JHalbauer commented Aug 7, 2024

r.dop.import downloads digital orthophotos (DOPs) for specified German federal state and area of interest, stores it in a local directory and creates a single file of DOPs in GRASS GIS.

jhalbauer and others added 30 commits June 27, 2022 17:21
r.dop.import: downloads and imports DOPs for NRW

See merge request mundialis/actinia-assets!186
r.dop.import: HE and BB

See merge request mundialis/actinia-assets!217
changed indentation (linting)

See merge request mundialis/actinia-assets!218
added BE working like BB for DOP import

See merge request mundialis/actinia-assets!219
r.dop.import tests

See merge request mundialis/actinia-assets!222
Optimize r dop import

See merge request mundialis/actinia-assets!209
changed download urls to latest

See merge request mundialis/actinia-assets!225
update downlaod urls

See merge request mundialis/actinia-assets!228
r.dop.import: fix Tindex

See merge request mundialis/actinia-assets!229
@neteler
Copy link
Member

neteler commented Aug 7, 2024

The Python formatting isn't yet ready for flake8.

Suggestion: use ruff (see latest ruff related PRs in the GRASS GIS project).

(see also: actinia-org/actinia-core#539)

@@ -5,3 +5,21 @@ on: [push]
jobs:
lint:
uses: mundialis/github-workflows/.github/workflows/linting.yml@main
with:
VALIDATE_HTML: false
ruff:
Copy link
Member

Choose a reason for hiding this comment

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

Vielleicht könntest du diesen Teil zum lining-github-workflows hinzufügen.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Soll das aber dann wirklich für alle Repos verwendet werden auch wenn es bspw. nicht GRASS ist? Und dann müsste man das auch bei allen Repos entsprechend anpassen.

.pylintrc Outdated Show resolved Hide resolved
@JHalbauer JHalbauer merged commit f820f92 into main Aug 21, 2024
10 checks passed
@JHalbauer JHalbauer deleted the r.dop.import branch August 21, 2024 11:53
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.

4 participants