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

broken WPS since 0.29.0 #893

Closed
fmigneault opened this issue Oct 16, 2023 · 1 comment · Fixed by #894
Closed

broken WPS since 0.29.0 #893

fmigneault opened this issue Oct 16, 2023 · 1 comment · Fixed by #894
Assignees
@tomkralidis
Labels
bug wps

Comments

@fmigneault
Copy link
Contributor

WPS is broken since e8dd55f, (ie: https://github.com/geopython/OWSLib/releases/tag/0.29.0)

This is because of changes like this one:

OWSLib/owslib/wps.py

Line 1614 in e8dd55f

LOGGER.debug(self.dataInputs[-1], prefix='\tInput: ')

e8dd55f#diff-9549cfa74c19d24dedb1e46b21f43c1a0b0986b018503074db64e219f08423deR1614

that replaced the dump function with prefix parameter, which logging functions don't have.

blocks crim-ca/weaver#577
error: https://github.com/crim-ca/weaver/actions/runs/6535582805/job/17745369444?pr=577
tomkralidis added a commit that referenced this issue Oct 16, 2023
@tomkralidis
fix LOGGER.debug statements (#893)
11723c5
@tomkralidis tomkralidis mentioned this issue Oct 16, 2023
Merged
@tomkralidis
Copy link
Member

PR in #894. Can you test?

tomkralidis added a commit that referenced this issue Oct 16, 2023
@tomkralidis
fix LOGGER.debug statements (#893) (#894)
5fad88b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tomkralidis tomkralidis

Labels
bug wps
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix LOGGER.debug statements (#893) geopython/OWSLib
2 participants
@tomkralidis @fmigneault