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

Option to disable printing error logs on failed job #505

Closed
soxofaan opened this issue Nov 15, 2023 · 1 comment
Closed

Option to disable printing error logs on failed job #505

soxofaan opened this issue Nov 15, 2023 · 1 comment
Assignees
@ElienVandermaesenVITO
Labels
feature request good first issue low hanging fruit

Comments

@soxofaan
Copy link
Member

We currently automatically print error logs when a job failed from start_and_wait or related.
Sometimes it's not necessary or useful to do this, so there should be an option to disable that feature

openeo-python-client/openeo/rest/job.py

Line 308 in bab3030

# TODO: allow to disable this printing logs (e.g. in non-interactive contexts)?

ElienVandermaesenVITO added a commit that referenced this issue Dec 16, 2024
@ElienVandermaesenVITO
issue #505 option to disable error logs in batch jobs
f732d74
Closed
ElienVandermaesenVITO added a commit that referenced this issue Dec 16, 2024
@ElienVandermaesenVITO
issue #505 option to disable error logs in batch jobs
6f3decb
Closed
soxofaan added a commit that referenced this issue Jan 7, 2025
@soxofaan
Issue #505 fintune #687
0700539 
- add changelog entry
- doc finetuning
- cover mlmodel and vectorcube too
- darker code style
- make test more to the point
soxofaan pushed a commit that referenced this issue Jan 7, 2025
@ElienVandermaesenVITO @soxofaan
issue #505 option to disable error logs in batch jobs
cf9e4ec
soxofaan added a commit that referenced this issue Jan 7, 2025
@soxofaan
Issue #505 fintune #687
bd9160c 
- add changelog entry
- doc finetuning
- cover mlmodel and vectorcube too
- darker code style
- make test more to the point
@soxofaan
Copy link
Member Author

soxofaan commented Jan 7, 2025

merged #687

@soxofaan soxofaan closed this as completed Jan 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ElienVandermaesenVITO ElienVandermaesenVITO

Labels
feature request good first issue low hanging fruit
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@soxofaan @jdries @ElienVandermaesenVITO