-
Notifications
You must be signed in to change notification settings - Fork 4
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
concert: Fix merge errors #531
base: master-with-decentralized
Are you sure you want to change the base?
concert: Fix merge errors #531
Conversation
Added |
@@ -91,27 +91,37 @@ def write_libtiff(file_name, data): | |||
return file_name | |||
|
|||
|
|||
def create_directory(directory, rights="750"): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It should be this, not 0o0750.
writer: Type[TiffWriter] = TiffWriter, | ||
prefix: str = "image_{:>05}.tif", | ||
start_index: int = 0, | ||
bytes_per_file=0) -> int: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Rights should make it back into the parameters.
@@ -125,7 +135,7 @@ def write_images(pqueue, writer=TiffWriter, prefix="image_{:>05}.tif", start_ind | |||
im_writer = writer(prefix, bytes_per_file, append=True) | |||
|
|||
if dir_name and not os.path.exists(dir_name): | |||
create_directory(dir_name, rights=rights) | |||
create_directory(dir_name) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We want the rights here as well.
@@ -369,7 +379,7 @@ async def __ainit__(self, | |||
|
|||
async def _descend(self, name: str) -> None: | |||
new = os.path.join(self._current, name) | |||
create_directory(new, rights=self._rights) | |||
create_directory(new) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
here too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Look at #528 regarding the rights. There I changed the int-based specification to string-based one, which you reverted here.
Premise: Fixes the errors introduced during merge of decentralized to master. Verified with session tests.