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

r.maxent.train: bugfix and clarification help text #1230

Merged
merged 6 commits into from
Oct 25, 2024

Conversation

ecodiv
Copy link
Contributor

@ecodiv ecodiv commented Oct 24, 2024

Fixes a bug and some small rewrites of help text

@ecodiv ecodiv added the bug Something isn't working label Oct 24, 2024
@ecodiv ecodiv self-assigned this Oct 24, 2024
@neteler
Copy link
Member

neteler commented Oct 24, 2024

There is an unexpected gs file included (38 MB, apparently PDF). Accidentally committed?

@ecodiv
Copy link
Contributor Author

ecodiv commented Oct 24, 2024

Yes, not sure how that happened, definitely unintentional. Thanks for catching that, I'll remove it and check when back behind the computer.

Removing gs file (still need to find out how it ended up there)
@ecodiv
Copy link
Contributor Author

ecodiv commented Oct 25, 2024

@neteler Removed the gs file. Now one of the test fails. From the error message, I can't really see what is wrong, and whether there is a problem with my code. Do you have an idea?

@neteler
Copy link
Member

neteler commented Oct 25, 2024

The CI says " 7 successful checks" - which error do you mean?

@ecodiv
Copy link
Contributor Author

ecodiv commented Oct 25, 2024

The CI says " 7 successful checks" - which error do you mean?

Sorry for the noise, there was an error, but running the checks again solved it, so apparently not related to my code/text.

@ecodiv ecodiv merged commit a71681b into OSGeo:grass8 Oct 25, 2024
7 checks passed
@ecodiv ecodiv deleted the r_maxent_train branch October 25, 2024 21:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants