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

jgrss/#242 #244

Merged
merged 9 commits into from
Mar 2, 2023
Merged

jgrss/#242 #244

merged 9 commits into from
Mar 2, 2023

Conversation

jgrss
Copy link
Owner

@jgrss jgrss commented Mar 2, 2023

What is this PR changing?

Following #242, this changes Path.rename to Path.replace in order to handle files in Windows.

@jgrss jgrss added the bug Something isn't working label Mar 2, 2023
@jgrss jgrss self-assigned this Mar 2, 2023
@jgrss
Copy link
Owner Author

jgrss commented Mar 2, 2023

@mmann1123 Can you do one more quick look? I added a test and deprecated an inconsistent argument.

@mmann1123
Copy link
Collaborator

@jgrss sure give me a bit

@mmann1123
Copy link
Collaborator

I am getting an error on coreg... but might have to do with my build.

======================================================================
ERROR: test_coreg_data (test_coreg.TestCOREG)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/geowombat/tests/test_coreg.py", line 29, in test_coreg_data
    data = gw.coregister(
  File "/geowombat/src/geowombat/core/sops.py", line 1230, in coregister
    raise NameError
NameError

======================================================================
ERROR: test_coreg_shift (test_coreg.TestCOREG)
Tests a 1-pixel shift.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/geowombat/tests/test_coreg.py", line 79, in test_coreg_shift
    shifted = gw.coregister(
  File "/geowombat/src/geowombat/core/sops.py", line 1230, in coregister
    raise NameError
NameError

======================================================================
ERROR: test_coreg_transform_data (test_coreg.TestCOREG)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/geowombat/tests/test_coreg.py", line 50, in test_coreg_transform_data
    data = gw.coregister(
  File "/geowombat/src/geowombat/core/sops.py", line 1230, in coregister
    raise NameError
NameError

----------------------------------------------------------------------
Ran 88 tests in 20.577s
FAILED (errors=3)

@jgrss
Copy link
Owner Author

jgrss commented Mar 2, 2023

I am getting an error on coreg... but might have to do with my build.

======================================================================
ERROR: test_coreg_data (test_coreg.TestCOREG)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/geowombat/tests/test_coreg.py", line 29, in test_coreg_data
    data = gw.coregister(
  File "/geowombat/src/geowombat/core/sops.py", line 1230, in coregister
    raise NameError
NameError

======================================================================
ERROR: test_coreg_shift (test_coreg.TestCOREG)
Tests a 1-pixel shift.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/geowombat/tests/test_coreg.py", line 79, in test_coreg_shift
    shifted = gw.coregister(
  File "/geowombat/src/geowombat/core/sops.py", line 1230, in coregister
    raise NameError
NameError

======================================================================
ERROR: test_coreg_transform_data (test_coreg.TestCOREG)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/geowombat/tests/test_coreg.py", line 50, in test_coreg_transform_data
    data = gw.coregister(
  File "/geowombat/src/geowombat/core/sops.py", line 1230, in coregister
    raise NameError
NameError

----------------------------------------------------------------------
Ran 88 tests in 20.577s
FAILED (errors=3)

Hmm, was that a new build? All tests just passed on this PR branch.

@jgrss jgrss mentioned this pull request Mar 2, 2023
1 task
@mmann1123
Copy link
Collaborator

@jgrss I think its just an issue with how I did the dockerfile.

updating testing environment 
note:  coreg isn't working we should try to figure out why
@jgrss jgrss merged commit 15b6f3f into main Mar 2, 2023
@jgrss jgrss deleted the jgrss/#242 branch March 2, 2023 18:47
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