Backtracking tests and moving modules around for better organization#36
Open
wli51 wants to merge 12 commits into
Open
Backtracking tests and moving modules around for better organization#36wli51 wants to merge 12 commits into
wli51 wants to merge 12 commits into
Conversation
…which internally serializes and deserializes the manifest, so for dataset to and from config the only source of truth would be the file_state attribute.
…implified serialization logic
…ual_stain_flow into dev-add-patch-dataset
…atial dimension behavior. Fix inappropriate property decorator on callable functions.
…, fix imports and tests
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR primes a series of future refactors whose collective goal would be improving readability and expand-ability of models, and ultimately laying foundation for future feature addition of allowing conditioning by a separate variable while computing convolutions on images/feature maps (conditioning is required for diffusion modelling).
This PR does the following:
blockssub package without changing behavior, adjust imports