Skip to content

Conversation

@schilling40
Copy link
Contributor

This pull request should establish the initial format and tools for the replication of block extraction from 3D volumes and the post-processing of SGNs.

WIP

@schilling40
Copy link
Contributor Author

I split the functionality of the block extraction formerly entirely localized within scripts/extract_block.py into the function being localized in flamingo_tools/extract_block.py, so that it is also accessible from other modules. Should I call such functions [...]_utils.py or what would be a good practice?

@constantinpape
Copy link
Contributor

I split the functionality of the block extraction formerly entirely localized within scripts/extract_block.py into the function being localized in flamingo_tools/extract_block.py, so that it is also accessible from other modules. Should I call such functions [...]_utils.py or what would be a good practice?

Yes, I think calling it something with util at the end make sense, maybe block_extraction_util.py? That sounds more natural.

Besides this, the PR looks good to merge from my end. So go ahead and merge it once it's ready from your end.

@schilling40 schilling40 merged commit 2a26409 into master Jun 10, 2025
6 checks passed
@schilling40 schilling40 deleted the repro_extraction_postprocessing branch June 10, 2025 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants