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.

Sign up for GitHub

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

Jump to bottom

Adaptions to run modules in gui #67

Merged
merged 18 commits into from
Jan 5, 2024
Merged

Adaptions to run modules in gui #67

merged 18 commits into from
Jan 5, 2024

Conversation

juleshaas
Copy link
Contributor

@juleshaas juleshaas commented Jan 3, 2024 •
edited
Loading

MR

  • makes changes to make modules usable in GUI
  • makes smaller label changes
  • updates black version
  • adds pandas to docker image
  • adds description of how to start docker on Windows
  • lints README

@juleshaas juleshaas changed the title Draft: adaptions to run modules in gui Adaptions to run modules in gui Jan 4, 2024
linakrisztian
linakrisztian approved these changes Jan 4, 2024
View reviewed changes
Copy link
Contributor

@linakrisztian linakrisztian left a comment

Choose a reason for hiding this comment

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

Just some small remarks

.github/workflows/super-linter.yml Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
grass-gis-addons/m.analyse.trees/v.trees.cd/v.trees.cd.py Show resolved Hide resolved
grass-gis-addons/m.analyse.trees/v.trees.param/v.trees.param.py Outdated Show resolved Hide resolved
juleshaas and others added 2 commits January 4, 2024 16:06
@juleshaas
remove debugging message
bb2075b
@anikaweinmann @linakrisztian
Apply suggestions from code review
e2ea1e9 
Co-authored-by: linakrisztian <106728040+linakrisztian@users.noreply.github.com>
@anikaweinmann anikaweinmann merged commit a53842d into main Jan 5, 2024
7 checks passed
@juleshaas juleshaas deleted the changes_for_gui branch January 5, 2024 07:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anikaweinmann anikaweinmann anikaweinmann approved these changes

@linakrisztian linakrisztian linakrisztian approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@juleshaas @anikaweinmann @linakrisztian